Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: verify JWT using JWK header #308

Merged
merged 1 commit into from
Jul 13, 2022
Merged

Conversation

DeepDiver1975
Copy link
Collaborator

@DeepDiver1975 DeepDiver1975 commented Jul 5, 2022

refs https://datatracker.ietf.org/doc/html/rfc7517
refs https://curity.io/resources/learn/self-contained-jwts/

List of common tasks a pull request require complete

  • Changelog entry is added or the pull request don't alter library's functionality

@azmeuk
Copy link
Collaborator

azmeuk commented Jul 8, 2022

Hi @DeepDiver1975
I think I won't have time to review this patch until monday. It looks quite short, if you are confident, maybe you can merge this now. If you don't, I will take time monday.

@DeepDiver1975
Copy link
Collaborator Author

There is no rush and I'd really welcome a review: more from the overall prespective - the code itself is rather simple ;-)

@DeepDiver1975 DeepDiver1975 merged commit e77e20f into master Jul 13, 2022
@DeepDiver1975 DeepDiver1975 deleted the feat/support-header-jwk branch July 13, 2022 10:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants