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.

Sign up for GitHub

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

Jump to bottom

Login via Azure AD #160

Open
rettichschnidi opened this issue Oct 10, 2024 · 3 comments · May be fixed by #162
Open

Login via Azure AD #160

rettichschnidi opened this issue Oct 10, 2024 · 3 comments · May be fixed by #162
Labels
enhancement New feature or request
Milestone
v0.5.0

Comments

@rettichschnidi
Copy link
Contributor

For enterprise users, logins should be possible using existing Azure AD infrastructure.
@b-rowan
Copy link
Contributor

b-rowan commented Oct 10, 2024

Looking into this, seems not too hard, but will def need some testing.

@easybe
Copy link
Collaborator

easybe commented Oct 10, 2024

Yes, SAML authentication would indeed be a nice to have.

@easybe easybe added the enhancement New feature or request label Oct 10, 2024
@b-rowan b-rowan linked a pull request Oct 10, 2024 that will close this issue
Add Azure/EntraID route for SAML #162
Draft
@b-rowan
Copy link
Contributor

b-rowan commented Oct 10, 2024

@easybe can you take a look at the PR I attached, it defines the framework for this, but I don't have a good way to test the auth itself. Specifically need to figure out what to do here -

goosebit/goosebit/auth/azure.py

Line 38 in 2b786d9

# TODO: Not sure how to use this exactly...

@tsagadar tsagadar added this to the v0.5.0 milestone Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.5.0
Development

Successfully merging a pull request may close this issue.

Add Azure/EntraID route for SAML UpstreamDataInc/goosebit
4 participants
@easybe @rettichschnidi @tsagadar @b-rowan