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

Add Azure/EntraID route for SAML #162

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add Azure/EntraID route for SAML #162

wants to merge 1 commit into from

Conversation

b-rowan
Copy link
Contributor

@b-rowan b-rowan commented Oct 10, 2024

Adds support for SAML via the optional azure extra. This can be configured with the config.auth.azure options.

Adds support for SAML via the optional `azure` extra.
This can be configured with the `config.auth.azure` options.
@b-rowan b-rowan linked an issue Oct 10, 2024 that may be closed by this pull request
@rettichschnidi
Copy link
Contributor

Wow, that was fast. Great, thanks a lot! However, I am not (yet) in a position to actually test this. We (@easybe, @tsagadar , worst case... I) will need to talk to our IT department first.

@b-rowan
Copy link
Contributor Author

b-rowan commented Oct 10, 2024

It's actually not even ready yet, it will need some fixing, I think the requests I'm trying from the msal lib are probably not right, I will need someone with an actual AzureAD account to try them out and add the proper handling. For now this is just a button, some settings, some checks, and dependency handling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Login via Azure AD
3 participants