-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add script to enable AAD Auth/Authz #404
Comments
I've enabled AAD for the other sample: I did it via preprovision and postprovision hooks, and some scripts to create the app registration. |
I would love a PR to this repo for those changes! |
@wherewith also if you could display the user icon in the headers with the logged in user details. |
I think that whatever solution was used on the other sample would be perfect for this repo as a starting point @pamelafox |
Yes please, it would be great to have some kind of authentication so not everyone can access the deploy. |
PR at #534 |
We do have documentation now about using AAD but we'd like to automate it with scripts. @mattmsft is going to extend my scripts to work with the document-level access control he just added. |
This is fantastic! I was hesitant to deploy the app initially and then manually integrate Azure AD later. Unfortunately, the official documentation didn't provide clear steps on how to achieve this using deployment scripts. |
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this issue will be closed. |
Sorry we never closed this - we did eventually do this |
This issue is for a: (mark with an
x
)Expected/desired behavior
With the help of AppService Auth enable Authentication and Authorization for specific AAD group in the tenant
The text was updated successfully, but these errors were encountered: