From ea5374979d83e198c2cb26e5a68fc574f92c8e99 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= <mail@3v1n0.net>
Date: Mon, 18 Nov 2024 19:35:41 +0100
Subject: [PATCH 1/2] dbusservice: Fix indentation on method arguments

---
 internal/dbusservice/dbusservice.go | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/internal/dbusservice/dbusservice.go b/internal/dbusservice/dbusservice.go
index b0131fe..357843e 100644
--- a/internal/dbusservice/dbusservice.go
+++ b/internal/dbusservice/dbusservice.go
@@ -15,16 +15,16 @@ const intro = `
 <node>
 	<interface name="%s">
 		<method name="NewSession">
-		<arg type="s" direction="in" name="username"/>
-		<arg type="s" direction="in" name="lang"/>
-		<arg type="s" direction="in" name="mode"/>
-		<arg type="s" direction="out" name="sessionID"/>
-		<arg type="s" direction="out" name="encryptionKey"/>
+			<arg type="s" direction="in" name="username"/>
+			<arg type="s" direction="in" name="lang"/>
+			<arg type="s" direction="in" name="mode"/>
+			<arg type="s" direction="out" name="sessionID"/>
+			<arg type="s" direction="out" name="encryptionKey"/>
 		</method>
 		<method name="GetAuthenticationModes">
-		<arg type="s" direction="in" name="sessionID"/>
-		<arg type="a{ss}" direction="in" name="supportedUILayouts"/>
-		<arg type="a{ss}" direction="out" name="authenticationModes"/>
+			<arg type="s" direction="in" name="sessionID"/>
+			<arg type="a{ss}" direction="in" name="supportedUILayouts"/>
+			<arg type="a{ss}" direction="out" name="authenticationModes"/>
 		</method>
 		<method name="SelectAuthenticationMode">
 			<arg type="s" direction="in" name="sessionID"/>

From ff546773a49a41ad507ff9ba1ede8d4ee03663db Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= <mail@3v1n0.net>
Date: Mon, 18 Nov 2024 19:36:03 +0100
Subject: [PATCH 2/2] dbusservice: Use array of strings maps arguments types
 for GetAuthenticationModes

The method accepts a slice of maps and returns another slice of maps, so
we should respect that in the introspection data.
---
 internal/dbusservice/dbusservice.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/internal/dbusservice/dbusservice.go b/internal/dbusservice/dbusservice.go
index 357843e..2159cbd 100644
--- a/internal/dbusservice/dbusservice.go
+++ b/internal/dbusservice/dbusservice.go
@@ -23,8 +23,8 @@ const intro = `
 		</method>
 		<method name="GetAuthenticationModes">
 			<arg type="s" direction="in" name="sessionID"/>
-			<arg type="a{ss}" direction="in" name="supportedUILayouts"/>
-			<arg type="a{ss}" direction="out" name="authenticationModes"/>
+			<arg type="aa{ss}" direction="in" name="supportedUILayouts"/>
+			<arg type="aa{ss}" direction="out" name="authenticationModes"/>
 		</method>
 		<method name="SelectAuthenticationMode">
 			<arg type="s" direction="in" name="sessionID"/>