From 08d9801829abf903104c299750b0b514cd4bc0ef Mon Sep 17 00:00:00 2001
From: Cameron Perera <cameron.perera@hashicorp.com>
Date: Mon, 29 Jan 2024 14:44:50 -0600
Subject: [PATCH] =?UTF-8?q?refactor:=20=F0=9F=92=A1=20add=20-addr=20flag?=
 =?UTF-8?q?=20to=20add-token=20command=20(#2118)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* refactor: 💡 add -addr flag to add-token command

* style: 💄 remove old comment
---
 ui/desktop/electron-app/src/services/client-daemon-manager.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/ui/desktop/electron-app/src/services/client-daemon-manager.js b/ui/desktop/electron-app/src/services/client-daemon-manager.js
index 649c34988..407203061 100644
--- a/ui/desktop/electron-app/src/services/client-daemon-manager.js
+++ b/ui/desktop/electron-app/src/services/client-daemon-manager.js
@@ -112,16 +112,15 @@ const addTokenCliCommand = (token) => {
   const addTokenCommand = [
     'daemon',
     'add-token',
+    `-addr=${runtimeSettings.clusterUrl}`,
     '-format=json',
     '-token=env://BOUNDARY_TOKEN',
     '-keyring-type=none',
   ];
   const sanitizedToken = sanitizer.base62EscapeAndValidate(token);
 
-  // TODO: Is setting an env var for the address the only way? I'm surprised to not see an -addr flag
   const { stdout, stderr } = spawnSync(addTokenCommand, {
     BOUNDARY_TOKEN: sanitizedToken,
-    BOUNDARY_ADDR: runtimeSettings.clusterUrl,
   });
   let parsedResponse = jsonify(stdout);