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) add refresh token tools #584

Closed
wants to merge 3 commits into from

Conversation

qbey
Copy link
Member

@qbey qbey commented Jan 28, 2025

Purpose

This provides the required "access token" and "refresh token" storage a refresh procedure to allow calls to "La Régie" resource server.

Proposal

  • allow access token storage in user session
  • allow refresh token encrypted storage in user session
  • create middleware for access token refresh and provide a decorator

@qbey qbey self-assigned this Jan 28, 2025
@qbey qbey requested a review from sampaccoud January 28, 2025 13:54
@qbey qbey force-pushed the qbey/oidc-token-storage-and-refresh branch 2 times, most recently from 97db241 to 35a2425 Compare January 30, 2025 10:09
@qbey qbey force-pushed the qbey/oidc-token-storage-and-refresh branch 3 times, most recently from e08d87d to 75d0fe5 Compare March 4, 2025 11:39
@qbey qbey force-pushed the qbey/oidc-token-storage-and-refresh branch from 75d0fe5 to a27e93c Compare March 18, 2025 08:36
qbey added 3 commits March 19, 2025 17:05
This provides a way to to refresh the OIDC access token.
The OIDC token will be used to request data to a resource server.
This code is highly related to
mozilla/mozilla-django-oidc#377
This code is not very clean but it reduces the footprint
to add refresh token storage in session.
This could be cleaned up once the PR it merged:
mozilla/mozilla-django-oidc#377
Enforce refresh token encryption for the session storage.
@qbey qbey force-pushed the qbey/oidc-token-storage-and-refresh branch from 62864df to f30c4ff Compare March 19, 2025 16:07
@qbey qbey requested a review from lunika March 20, 2025 14:27
@qbey
Copy link
Member Author

qbey commented Apr 2, 2025

Will be replaced by an integration of suitenumerique/django-lasuite#3

@qbey
Copy link
Member Author

qbey commented Apr 3, 2025

Superseeded by #839 : the new dependency will provide this feature.

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

Successfully merging this pull request may close these issues.

2 participants