Skip to content

fix: Ensure required claims applies to custom claims#423

Merged
simoneb merged 1 commit intonearform:masterfrom
kieranm:custom-required-claims
Feb 15, 2024
Merged

fix: Ensure required claims applies to custom claims#423
simoneb merged 1 commit intonearform:masterfrom
kieranm:custom-required-claims

Conversation

@kieranm
Copy link
Copy Markdown
Contributor

@kieranm kieranm commented Feb 15, 2024

This PR ensures that custom claims are checked if they are specified in requiredClaims. Previously only a small subset of claims were validated.

Fixes #422

@kieranm kieranm changed the title Ensure required claims applies to custom claims fix: Ensure required claims applies to custom claims Feb 15, 2024
Copy link
Copy Markdown
Member

@simoneb simoneb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks 🚀

@simoneb simoneb merged commit 3409d1d into nearform:master Feb 15, 2024
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.

requiredClaims: Not all claims are checked

2 participants