Merge pull request #348 from blinqas/fix-issue-github-federated-identity

Fix missing ARM_CLIENT_ID and ARM_TENANT_ID vars who is required when…
aztfmod · Dec 19, 2023 · 6a2576d · 6a2576d
2 parents fecd567 + 17a8b5b
commit 6a2576d
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/scripts/functions.sh b/scripts/functions.sh
@@ -294,8 +294,13 @@ function verify_azure_session {
     if [ "$session" == '' ]; then
         display_login_instructions
         error ${LINENO} "you must login to an Azure subscription first or 'rover login' again" 2
+    else
+        # If the session is a OIDC service principal we need to export the ARM variables
+        if [ "${ARM_USE_OIDC}" == "true" ]; then
+            export ARM_CLIENT_ID=$(echo $session | jq -r '.user.name')
+            export ARM_TENANT_ID=$(echo $session | jq -r '.tenantId')
+        fi
     fi
-
 }
 
 function login_as_sp_from_keyvault_secrets {