-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Azure authentication #2857
base: main
Are you sure you want to change the base?
Azure authentication #2857
Conversation
This reverts commit b9b5cd6.
…g more changes on app.py...
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
Thanks for the PR! It looks huge. Will take some time after discussion with @victordibia okay? |
Sounds good, thank you! I had talk to @victordibia about this some time back. Looking forward to your feedback. |
Why are these changes needed?
This is a sample of how azure authentication can be implemented. Leveraging azure authentication flow and app registration with roles. It also enables multi-tenant functionality by grouping objects by role/group.
Related issue number
resolves #2733
Checks