Version warning with output set to none
- 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