This repository was archived by the owner on Jun 23, 2023. It is now read-only.
OIDC Certification and FAPI profile#158
Merged
peppelinux merged 14 commits intomasterfrom Nov 22, 2021
Merged
Conversation
allow passing already hashed IDs from backends or hashing in a microservice instead
feat: add RawID to session manager
So one step back for the time being.
Don't specify cryptojwt requirement.
Changes that this PR accomplish: - If response_type == id_token return userinfo that normally would be returned from the userinfo endpoint in the id_token. - Don't run parse_cookie twice. One after the other. - Allow for different claims release policies to be in effect simultaneously (typically id_token+userinfo). - Error response when response_mode == form_post not handled correctly. return_uri needed. - Use extra info in ValueError exception instance to return correct error code. - Allow endpoints to have their own request verification error handler. - Having session ID in ID Token instance simplifies things. - Finding the last issued token of a specific type is useful. - Should check if client session is revoked. - Revoked all grants issued to a user/client pair. - Merged the two ClaimsInterface versions. - Removed tests using method that was removed and incomplete test. - Client registration endpoint should return a 201 HTTP response code on successful registration. - Default token lifetime should not be 0 (zero). Changed to be 30 minutes (1800 seconds). - Authorization error response MUST contain 'state' if it is present in the request. - Check cookie age. - Revoke tokens that has been minted using a code that then is used once more. - prompt==login forces re-authentication. - prompt attribute is a list. - FAPI OP example project
rohe
approved these changes
Nov 22, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes that this PR accomplish: