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

Handle JWT response data in userinfo endpoint #85

Merged

sergei-maertens merged 1 commit into master from feature/83-jwt-userinfo-response

Feb 7, 2024

Member

sergei-maertens commented Feb 6, 2024 •

edited

Loading

Closes #83

sergei-maertens requested a review from stevenbal

February 6, 2024 13:00

sergei-maertens commented

View reviewed changes

docker/import/test-realm.json Show resolved Hide resolved

sergei-maertens commented

View reviewed changes

mozilla_django_oidc_db/admin.py Outdated Show resolved Hide resolved

sergei-maertens commented

View reviewed changes

mozilla_django_oidc_db/jwt.py Show resolved Hide resolved

Member Author

sergei-maertens commented Feb 6, 2024

After this PR I want to go over the code again and update the type annotations to use friendlier syntax since Python 3.10

sergei-maertens commented

View reviewed changes

mozilla_django_oidc_db/typing.py Outdated Show resolved Hide resolved

sergei-maertens commented

View reviewed changes

mozilla_django_oidc_db/models.py Outdated Show resolved Hide resolved

sergei-maertens commented

View reviewed changes

mozilla_django_oidc_db/models.py Outdated Show resolved Hide resolved

sergei-maertens force-pushed the feature/83-jwt-userinfo-response branch 2 times, most recently from 64d6193 to 3998c42 Compare

February 6, 2024 13:09

Viicos approved these changes

View reviewed changes

.github/workflows/ci.yml Outdated Show resolved Hide resolved

docker/README.md Outdated Show resolved Hide resolved

mozilla_django_oidc_db/mixins.py Outdated Show resolved Hide resolved

mozilla_django_oidc_db/mixins.py Outdated Show resolved Hide resolved

mozilla_django_oidc_db/backends.py Outdated Show resolved Hide resolved

mozilla_django_oidc_db/jwt.py Outdated Show resolved Hide resolved

Member Author

sergei-maertens commented Feb 7, 2024

I've processed a bunch of feedback via #86 so that I can pull apart the changesets into more atomic commits


          ✨ [#83] Implement JWT user-info response data processing

e1b6b12

sergei-maertens force-pushed the feature/83-jwt-userinfo-response branch from 3998c42 to e1b6b12 Compare

February 7, 2024 14:07

sergei-maertens marked this pull request as ready for review

February 7, 2024 14:16

sergei-maertens merged commit 7ffe295 into master

8 checks passed

sergei-maertens deleted the feature/83-jwt-userinfo-response branch

February 7, 2024 14:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet