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

Option to allow the OIDC id_token to be stored in cookie and used as principal #3552

Closed
wants to merge 1 commit into from
Closed

Conversation

jmntn2000
Copy link
Contributor

Some OIDC providers don't return the actual profile in the access_token. This option will allow the id_token from the token endpoint to be stored in the cookie and utilized to build the principal for use in context.

@tomas-langer
Copy link
Member

Hello, this conflicts with existing PR: #3456
which allows usage of id token in cookie for logout purposes. This also adds encryption support

@jmntn2000
Copy link
Contributor Author

Hello, this conflicts with existing PR: #3456 which allows usage of id token in cookie for logout purposes. This also adds encryption support

Sorry I didn't realize the logout PR covered this. It doesn't cover the use case of using it as the Principal though. Would you be open to integrating that portion into your PR? I could assist if needed.

@jmntn2000
Copy link
Contributor Author

Closing due to conflict, will add ability to use id token as principal with changes in #3456

@jmntn2000 jmntn2000 closed this Oct 29, 2021
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.

2 participants