-
Notifications
You must be signed in to change notification settings - Fork 3.4k
logger.warning() doesnt mention required scope argument for granting permission #25982
Copy link
Copy link
Open
Labels
Auto-AssignAuto assign by botAuto assign by botAzure CLI TeamThe command of the issue is owned by Azure CLI teamThe command of the issue is owned by Azure CLI teamGraph(doesn't work with label-triggered comments; use Graph.Microsoft instead) az ad(doesn't work with label-triggered comments; use Graph.Microsoft instead) az adact-identity-squadquestionThe issue doesn't require a change to the product in order to be resolved. Most issues start as thatThe issue doesn't require a change to the product in order to be resolved. Most issues start as that
Milestone
Metadata
Metadata
Assignees
Labels
Auto-AssignAuto assign by botAuto assign by botAzure CLI TeamThe command of the issue is owned by Azure CLI teamThe command of the issue is owned by Azure CLI teamGraph(doesn't work with label-triggered comments; use Graph.Microsoft instead) az ad(doesn't work with label-triggered comments; use Graph.Microsoft instead) az adact-identity-squadquestionThe issue doesn't require a change to the product in order to be resolved. Most issues start as thatThe issue doesn't require a change to the product in order to be resolved. Most issues start as that
Describe the bug
The warning in the console gives a command that doesn't work without additional arguments.
Command Name
az ad app permission addErrors:
The output says:
"Invoking
az ad app permission grant --id {}--api 00000003-0000-0000-c000-000000000000is needed to make the change effective"However, if you run that, you'll get an error due to missing argument:
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.
az ad app permission add --id {} --api {} --api-permissions {}Expected Behavior
I expect it to give me a working command or at least to mention scope is needed.
Environment Summary
Additional Context
The line of code generating the warning is
azure-cli/src/azure-cli/azure/cli/command_modules/role/custom.py
Line 908 in 36786c5