Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace office365, azuread with updated azure provider #427

Closed
hueniverse opened this issue Sep 13, 2019 · 3 comments
Closed

Replace office365, azuread with updated azure provider #427

hueniverse opened this issue Sep 13, 2019 · 3 comments
Assignees
Milestone

Comments

@hueniverse
Copy link
Member

@hueniverse hueniverse commented Sep 13, 2019

No description provided.

@hueniverse hueniverse added this to the 10.1.2 milestone Sep 13, 2019
@hueniverse hueniverse self-assigned this Sep 13, 2019
@teddmason

This comment has been minimized.

Copy link

@teddmason teddmason commented Sep 17, 2019

@hueniverse Hi Eran, this change has broken our azureAD authentication, the microsoft graph api (https://graph.microsoft.com/v1.0/me) doesn't expose user application roles, and my provider has said that the graph api is locked down in production so is a non starter for us.

Is there a reason for ditching the old endpoint https://login.microsoftonline.com/{tenantId}/openid/userinfo ?

(edit: removing our tenantId from openid url)

@hueniverse

This comment has been minimized.

Copy link
Member Author

@hueniverse hueniverse commented Sep 17, 2019

Which provider do you want to see reintroduced? I can add it as -legacy as long as it is not deprecated by MS. Also, please open this as a new issue.

@teddmason

This comment has been minimized.

Copy link

@teddmason teddmason commented Sep 17, 2019

azuread is what we use in v10 and it is not being deprecated... as far as I'm aware, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.