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

WIP: TECH-676 Support cookieless session lookup, including logout #7

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

flipscholtz
Copy link

@flipscholtz flipscholtz commented Oct 30, 2024

  • Add new cookies.enableCookielessFallback config value, which causes sessions to be looked up from the jti in a parsed ID token passed via query params or form body (id_token_hint for the RP-initiated logout case).
  • Add the session ID to the jti field of the issued ID token so it can be picked up by the above lookup.
  • Add a separate cookies.doNotSet config value which will cause the library not to set any cookies at all. This can be used to avoid browser alerts about third-party cookies when running as an iframe.
  • Add a bypassConsent config flag for the rpInitiatedLogout feature, which makes the server not ask for logout consent, for cases where logout consent is managed in a separate UI.

Corresponding civic-auth WIP PR here

TODO:

  • Write tests for the new functionality
  • Update README and other docs

… allows sessions to be looked up from an id_token_hint query parameter, and interactions from the UID in path params, if session cookies are not found. A separate 'cookies.doNotSet' option will cause cookies not to be set at all. The former is used in iframe environments where browsers might not send cookies even if they are set.

Also a add a 'bypassConsent' parameter to the rpInitiatedLogout feature, which will bypass asking the user for logout consent if true.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant