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

Update the Azure AD App Registration's process guide #81

Closed
magatz opened this issue Aug 30, 2019 · 7 comments
Closed

Update the Azure AD App Registration's process guide #81

magatz opened this issue Aug 30, 2019 · 7 comments

Comments

@magatz
Copy link

magatz commented Aug 30, 2019

The registration process on Aif an app on Azure has changed so much that the guide is not so useful anymore.
I'm trying to figure out the new needed steps, and any help is welcome.

@jobec
Copy link
Collaborator

jobec commented Oct 11, 2019

Azure did it again? They move so fast that it's hard to keep up with them.

Same with what they did in #67

My experience with Azure sadly is limited, but I'll see if I can figure it out (again) how to get an app registered.

If anyone reading this has more insight into what changed, feel free to open a MR to update the docs.

@dVerhees
Copy link

dVerhees commented Mar 6, 2020

@magatz
I've actually managed to set it up with the latest Azure AD. The main difference is that when creating the web application registration you need to manually expose an API and select a scope for the API.
In addition the names have changed from WEB/API to just WEB application. and Native to Public client/native (mobile & desktop)

The api will also use a different url, like: api://application_id

I've used the following scope settings to get it to work:
Scope Name: Name for the scope
Who can consent? : Admins and users
Admin consent display name: This will show up if an admin logs in and needs to consent
Admin consent description: Same as above.

You won't normally see those consent messages as you'll use the "grant admin consent" option on the public client/native application registration.

@jobec I will probably be able to provide you with screenshots but I will need to blur all sensitive information so I don't know how helpful they will be.

@zdrouse
Copy link

zdrouse commented Aug 6, 2020

Having some issues getting set up with Azure AD. Any update for changes to make with app registrations?

Been trying to follow the guide here but it looks a lot different on Azure side now.
https://django-auth-adfs.readthedocs.io/en/latest/azure_ad_config_guide.html

Additionally, it's not entirely clear to me why in Azure there are 2 app registrations needed for Web and Native. Would be helpful to better understand that part.

@jobec
Copy link
Collaborator

jobec commented Sep 13, 2020

About the 2 registrations, that's because Django is acting as both "Client" and "Resource Server".
See here for more info: https://django-auth-adfs.readthedocs.io/en/latest/oauth2_explained.html

If I find dome time, I'll try to re-document the AzureAD stuff. But Like I said, they move so fast that it results in having to start from scratch every 6 months or so.

@gunflap
Copy link

gunflap commented Sep 23, 2020

@zdrouse @jobec @magatz @dVerhees I'm having some issues with Azure AD as well (django_auth_adfs Invalid issuer) - unsure if I'm setting things up correctly with Azure (since docs are outdated). Could you drop your photos/configuration (specifically AUTH_ADFS and your settings on Azure) for reference? Thank you.

@JonasKs
Copy link
Member

JonasKs commented Jan 18, 2021

#126 has been merged with updated docs for Azure AD! 🎉

As for how you set up a front end application (such as Vue or react) to speak with backend will come some time soon.

I'll keep this issue open until then.

@JonasKs
Copy link
Member

JonasKs commented Aug 27, 2021

This has been done.

@JonasKs JonasKs closed this as completed Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants