search

MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
10.22k stars 21.38k forks source link

German cloud support for Azure AD v2.0 #28661

Closed nfegan closed 5 years ago

nfegan commented 5 years ago

This page suggests that Azure AD v2.0 is only supported for the global service. Our testing suggests that it seems to work against German cloud. Wondering now if it's now supported, but this page wasn't updated?

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

shashishailaj commented 5 years ago

@nfegan Thank you for your feedback . We will investigate and update

SaurabhSharma-MSFT commented 5 years ago

@nfegan I do not think that it is currently supported. Can you please provide steps with the authorization endpoint details on how exactly you have tested this.

mbfgit commented 5 years ago

Hi @SaurabhSharma-MSFT, I work with @nfegan

We're using https://login.microsoftonline.de/common/oauth2/v2.0/authorize to get the authorisation code and https://login.microsoftonline.de/common/oauth2/v2.0/token for code redemption / token refresh.

We had to prepend https://graph.microsoft.de/ to the scopes we were asking for in those requests - e.g. https://graph.microsoft.de/Files.Read.All

We can then call https://graph.microsoft.de/v1.0/me/drive for example with the access token we've received and it works.

SaurabhSharma-MSFT commented 5 years ago

@nfegan Ok. I have assigned this issue to content author to investigate and update the document as appropriate.

rayluo commented 5 years ago

To add more info into this conversation, I also noticed there is actually another official document page here mentioning "use MSAL library in National Cloud", and we know that MSAL talks to AADv2 endpoint only. So, that hints National Cloud does support AAD v2.0 endpoints.

negoe commented 5 years ago

@nfegan v2.0 endpoints are supported on National Clouds but MSA scenarios are not supported. Here is PR to update the documentation.

CelesteDG commented 5 years ago

Documentation update has been merged to master (see linked PR). Thank you, @negoe :) This update will go live tomorrow, ~10 AM Pacific.

please-close