Skip to content

Conversation

@HEllRZA
Copy link
Contributor

@HEllRZA HEllRZA commented Jul 31, 2020

This PR adds support for c_hash on the id_token.
The c_hash is required in the hybrid flow, and optional otherwise.

According to https://openid.net/specs/openid-connect-core-1_0.html#CodeValidation

If the ID Token is issued from the Authorization Endpoint with a code, which is the case for the response_type values code id_token and code id_token token, this is REQUIRED; otherwise, its inclusion is OPTIONAL.

OpenID Connect Certification
The test (OP-IDToken-c_hash) now succeeds

* fixed name collision (renamed hash->hashFunc)
@HEllRZA HEllRZA closed this Jul 31, 2020
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.

1 participant