fix: Make amr
claim an array to match the OIDC spec
#625
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.
This change makes
amr
claim follow the spec and be of an array type.See: https://openid.net/specs/openid-connect-core-1_0.html#IDToken
My team noticed that when trying to use
amr
claim with Hydra.Will follow up with Hydra PR to add support for
amr
claim as it is being dropped right now if used. Related issue: ory/hydra#1756Previous change: #401
Checklist
and signed the CLA.
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got green light (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
appropriate).
Further comments
This is breaking change, migration path is to wrap single values into the
[]string
slice and use a different field.