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

Support basic auth when authenticating #80

Closed
sergei-maertens opened this issue Feb 5, 2024 · 1 comment · Fixed by #82
Closed

Support basic auth when authenticating #80

sergei-maertens opened this issue Feb 5, 2024 · 1 comment · Fixed by #82
Assignees
@sergei-maertens

Comments

@sergei-maertens
Copy link
Member

Some OIDC providers require the client ID + secret to be sent through HTTP Basic Auth.
@sergei-maertens sergei-maertens self-assigned this Feb 5, 2024
sergei-maertens added a commit that referenced this issue Feb 5, 2024
@sergei-maertens
🐳 [#80] Set up infra + test for live integration test
b641a81
@sergei-maertens
Copy link
Member Author

A fix for mozilla/mozilla-django-oidc#517 is also needed.

sergei-maertens added a commit that referenced this issue Feb 5, 2024
@sergei-maertens
✅ [#80] Add assertion for default settings
e76cd0b 
* No Basic auth header
* Client ID and secret in request body
sergei-maertens added a commit that referenced this issue Feb 5, 2024
@sergei-maertens
✨ [#80] Support configuring basic auth for token endpoint
28e596f 
Certain OIDC providers require the client credentials to be sent
in the Basic Auth request header rather than in the request
body. This is now configurable in the admin.
sergei-maertens added a commit that referenced this issue Feb 5, 2024
@sergei-maertens
✨ [#80] Support configuring basic auth for token endpoint
e240343 
Certain OIDC providers require the client credentials to be sent
in the Basic Auth request header rather than in the request
body. This is now configurable in the admin.
@sergei-maertens sergei-maertens mentioned this issue Feb 5, 2024
Merged
sergei-maertens added a commit that referenced this issue Feb 6, 2024
@sergei-maertens
✨ [#80] Support configuring basic auth for token endpoint
fb09fff 
Certain OIDC providers require the client credentials to be sent
in the Basic Auth request header rather than in the request
body. This is now configurable in the admin.
sergei-maertens added a commit that referenced this issue Feb 6, 2024
@sergei-maertens
Merge pull request #82 from maykinmedia/feature/80-basic-auth-setting
b589bfe 
✨ [#80] Support configuring basic auth for token endpoint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sergei-maertens sergei-maertens

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

✨ [#80] Support configuring basic auth for token endpoint maykinmedia/mozilla-django-oidc-db
1 participant
@sergei-maertens