Azure/login should logout the active account at the beginning (#376)

* logout at the beginning

* remove logout step in test case
This commit is contained in:
Shiying Chen
2023-11-27 17:21:50 +08:00
committed by GitHub
parent 34b958dce7
commit 40935f9fb5
2 changed files with 7 additions and 5 deletions

View File

@@ -347,11 +347,6 @@ jobs:
script: |
core.setFailed('Last action should fail but not. Please check it.')
# logout first to avoid the conflict with SP1
- name: Azure CLI logout
run: |
az logout
# SP1 is ignored and SP2 will be used for login, but it will fail since SP2 has no access to the given subscription
- name: Login with both creds and individual parameters
id: login_12

View File

@@ -34,6 +34,13 @@ export class AzureCliLogin {
await this.executeAzCliCommand(["--version"], true, execOptions);
core.debug(`Azure CLI version used:\n${output}`);
try {
await this.executeAzCliCommand(["logout"], true, execOptions);
}
catch (error) {
core.debug(`Ignore logout error: "${error}"`);
}
this.setAzurestackEnvIfNecessary();
await this.executeAzCliCommand(["cloud", "set", "-n", this.loginConfig.environment], false);