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

OIDC #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

OIDC #2

wants to merge 8 commits into from

Conversation

Qup42
Copy link

@Qup42 Qup42 commented Jan 5, 2023

Configure custom scopes and use OIDC endpoints in Janus introduced by smartlgt/django-janus#7. Some code cleanup. Use Authorization header instead of Get Parameter.

@Qup42
Copy link
Author

Qup42 commented Jan 12, 2023

This is now mostly finished. I will deploy this (django-janus and janus-allauth-provider; entirety of OIDC changes) to one or two of our internal apps. If no problems occur I will mark this PR as ready to merge.

@Qup42
Copy link
Author

Qup42 commented Jan 13, 2023

I am now also working on storing the JWT Tokens. I have now found a solution to store the tokens that I am satisfied with. So the OIDC RP Initiated Logout may also happen sometime in the future.

@Qup42 Qup42 marked this pull request as ready for review January 15, 2023 18:48
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.

1 participant