-
Notifications
You must be signed in to change notification settings - Fork 118
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 support for keycloak>v4 & Django 4.1.1 #70
base: master
Are you sure you want to change the base?
Conversation
dabocs
commented
Oct 6, 2022
•
edited
edited
- Added support for keycloak > v17 & Django 4.1.1 (should be Django > v2.0)
- Fixed issues
- No access_token provided to compare against at_hash claim. #57
- Invalid Scopes with keycloak provider oauth2-proxy/oauth2-proxy#1448
- Updated steps at documentation to fix issue JWTClaimsError at /keycloak/login-complete #18 based on the answer https://stackoverflow.com/questions/53550321/keycloak-gatekeeper-aud-claim-and-client-id-do-not-match/53627747#53627747
- This package depends on upgrade PR for KC > v17
@Peter-Slump |
Hey @dabocs thank you for this PR. Unfortunately as you can understand it seems that @Peter-Slump does not maintain this project any longer. My recommendation, would be to fork this project using a new name, i.e django-keycloak2, apply your PR and release a new pypi version under the new name (if you are willing of course to commit to ths burden). Then we can add comments to the corresponding issues pointing to the forked project so users can use it isntead of this. Thank you |
@spapas Thanks for your advice. I already did that and forked the repo here |
Thank you very much @dabocs ! |