Update coder-login module to use environment variables

Changed from template variables to environment variables for CODER_USER_TOKEN
and CODER_DEPLOYMENT_URL to improve security and flexibility.

Co-authored-by: M Atif Ali <[email protected]>

Author: blink-so[bot]

registry/coder/modules/coder-login/main.tf

@@ -19,13 +19,13 @@ data "coder_workspace_owner" "me" {}
 
 resource "coder_script" "coder-login" {
   agent_id = var.agent_id
-  script = templatefile("${path.module}/run.sh", {
-    CODER_USER_TOKEN : data.coder_workspace_owner.me.session_token,
-    CODER_DEPLOYMENT_URL : data.coder_workspace.me.access_url
-  })
+  script = templatefile("${path.module}/run.sh", {})
   display_name       = "Coder Login"
   icon               = "/icon/coder.svg"
   run_on_start       = true
   start_blocks_login = true
-}
-
+  env = {
+    CODER_USER_TOKEN     = data.coder_workspace_owner.me.session_token
+    CODER_DEPLOYMENT_URL = data.coder_workspace.me.access_url
+  }
+}

registry/coder/modules/coder-login/run.sh

@@ -9,7 +9,7 @@ printf "$${BOLD}Logging into Coder...\n\n$${RESET}"
 
 if ! coder list > /dev/null 2>&1; then
   set +x
-  coder login --token="${CODER_USER_TOKEN}" --url="${CODER_DEPLOYMENT_URL}"
+  coder login --token="$CODER_USER_TOKEN" --url="$CODER_DEPLOYMENT_URL"
 else
   echo "You are already authenticated with coder."
 fi