Version warning with output set to none

[daniloesk]

According to the documentation it is a security recommendation to set AZURE_CORE_OUTPUT variable to "none".
But by doing so a "Failed to parse Azure CLI version" warning is recorded due to the block:

login/src/Cli/AzureCliLogin.ts

Lines 32 to 39 in 833e823

	await this.executeAzCliCommand(["version"], true, execOptions);
	core.debug(`Azure CLI version used:\n${output}`);
	try {
	this.azVersion = JSON.parse(output)["azure-cli"];
	}
	catch (error) {
	core.warning("Failed to parse Azure CLI version.");
	}

Since this command requires the JSON output it could have an explicit json output format argument. It should be safe since the ouput is captured by the script.
Or perhaps the warning could be changed into a debug message.

[daniloesk]

Taking a deeper look, this value is used later to change the behavior to CLI versions prior to 2.69.

login/src/Cli/AzureCliLogin.ts

Line 125 in 833e823

if (azcliMinorVersion < 69) {

So probably the output format should be forced as JSON to avoid errors on such versions.

[daniloesk]

As workaround, keep your global AZURE_CORE_OUTPUT set to 'none' but include a specific env with json only for the login step. Of course you can also use an env in every azure cli step.

env:
  AZURE_CORE_OUTPUT: none
jobs:
  example-job:
    steps:
      - name: Azure login
        env:
          AZURE_CORE_DEBUG: json
        uses: azure/login@v3
        with:
          creds: *****
      - name: Azure command
        run: az *****