Azure / Azure/login

Version warning with output set to none

Open Beginner friendly
#605 2 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
552
Forks
440
Avg merge
3d 5h
Merged PRs (30d)
18

Description

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:
https://github.com/Azure/login/blob/833e82333ca8452a40084fee403f7f16e65cba70/src/Cli/AzureCliLogin.ts#L32-L39

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with src/Cli/AzureCliLogin.ts lines 32-39 and reproduce the version check with AZURE_CORE_OUTPUT set to "none". Confirm how the Azure CLI version command is invoked and whether its output is parsed there. Done means the check still obtains the version without recording a warning when output is disabled globally.

Written by the indexing model from the issue text.

Assessment

Tech stack
azure, github-actions, typescript
Domain
ci-cd, cloud
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.