Merge pull request #212 from quexten/fix/client-credentials

Attempt to fix credentials not being set in gui
quexten · May 11, 2024 · 08f419a · 08f419a
2 parents e913180 + 9fbf1a0
commit 08f419a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/gui/src/gui/login.py b/gui/src/gui/login.py
@@ -52,9 +52,9 @@ def on_login(self):
         server = self.server_row.get_text()
         goldwarden.set_url(server)
         if client_id != "":
-            goldwarden.set_client_id(client_id)
+            print("set client id result", goldwarden.set_client_id(client_id.strip()))
         if client_secret != "":
-            goldwarden.set_client_secret(client_secret)
+            print("set client secret result", goldwarden.set_client_secret(client_secret.strip()))
         goldwarden.login_with_password(email, "")
         self.window.close()
 

diff --git a/gui/src/services/goldwarden.py b/gui/src/services/goldwarden.py
@@ -81,10 +81,10 @@ def get_environment():
         return None
 
 def set_client_id(client_id):
-    send_authenticated_command(f"config set-client-id \"{client_id}\"")
+    return send_authenticated_command(f"config set-client-id {client_id}")
 
 def set_client_secret(client_secret):
-    send_authenticated_command(f"config set-client-secret \"{client_secret}\"")
+    return send_authenticated_command(f"config set-client-secret {client_secret}")
 
 def login_with_password(email, password):
     result = send_authenticated_command(f"vault login --email {email}")