Skip to content

fix: capture missingRequiredClaim errors and map to 401 #369

Merged
gurgunday merged 3 commits intofastify:mainfrom
frederikprijck:fix/requiredClaims
Mar 12, 2025
Merged

fix: capture missingRequiredClaim errors and map to 401 #369
gurgunday merged 3 commits intofastify:mainfrom
frederikprijck:fix/requiredClaims

Conversation

@frederikprijck
Copy link
Copy Markdown
Contributor

@frederikprijck frederikprijck commented Mar 5, 2025

Fixes #368

Checklist

Signed-off-by: Frederik Prijck <frederik.prijck@gmail.com>
Signed-off-by: Frederik Prijck <frederik.prijck@gmail.com>
Signed-off-by: Frederik Prijck <frederik.prijck@gmail.com>
@frederikprijck frederikprijck marked this pull request as ready for review March 5, 2025 23:17
@frederikprijck
Copy link
Copy Markdown
Contributor Author

frederikprijck commented Mar 10, 2025

Thanks for approving all 🙏 .

Just wanted to follow up, is there anything needed from me to get this merged?

@gurgunday
Copy link
Copy Markdown
Member

Thanks for approving all 🙏 .

Just wanted to follow up, is there anything needed from me to get this merged?

No it's fine

@gurgunday gurgunday merged commit 523e752 into fastify:main Mar 12, 2025
11 checks passed
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.

When FAST_JWT_MISSING_REQUIRED_CLAIM is thrown, a 500 is returned and not a 401

4 participants