Azure / azure-cli

Azure Command-Line Interface
MIT License
4.01k stars 2.98k forks source link

Add redirectURI for single page application deployment in azure using Microsoft Graph API and Az rest command. #21668

Open abi-kannan opened 2 years ago

abi-kannan commented 2 years ago

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug Get error in deployment when adding SPA redirectURIs using microsoft Graph API. command used: az rest --method PATCH --uri 'https://graph.microsoft.com/v1.0/applications/' --headers 'Content-Type=application/json' --body '{"spa":{"redirectUris":["https:"]}}' error:

[error]WARNING: The underlying Active Directory Graph API will be replaced by Microsoft Graph API in a future version of Azure CLI. Please carefully review all breaking changes introduced during this migration: https://docs.microsoft.com/cli/azure/microsoft-graph-migration

this is causing the deployment to fail. I get this error intermittently.

To Reproduce Add redirecturi to single page application using az command and deploy using azure devops pipeline.

Expected behavior I am already using Microsoft Graph DB and redirectURIs instead of replyurl as mentioned in error. So I except deployment should completed without any warnings and error.

Environment summary I am deploying using azure devops and default az version.

yonzhan commented 2 years ago

@jiasli for awareness

jiasli commented 2 years ago

WARNING: The underlying Active Directory Graph API will be replaced by Microsoft Graph API in a future version of Azure CLI. Please carefully review all breaking changes introduced during this migration: https://docs.microsoft.com/cli/azure/microsoft-graph-migration

This warning is only added for az ad and az role commands (https://github.com/Azure/azure-cli/pull/21302). The warning won't be shown for az rest. It is also only a warning, not an error.

If you see any error with az rest, please share the corresponding error message.

spa support in az ad app create/update is currently not included in the Microsoft Graph migration:

We will consider this as a feature request.

ghost commented 2 years ago

👍 for spa support request.