-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modified JWK to reflect spec #60
Conversation
On hold, tests are broken inside did-endpoint, fixing... |
Ready for review again 👍🏻 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your formatter seems unrelated to cargo fmt
. I think we need to sync on this, as there are currently too many changes, which makes it hard to track the actual contribution of this PR.
@IngridPuppet I tracked down the problem. I am using https://github.com/jinxdash/prettier-plugin-rust inside my VS Code. I will disable it. |
Formating makes review tough. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do we proceed with code formating.
Closed and opened a new, cleaner: #70 |
…spec Clean: Modified JWK to reflect spec #60
Closes #49
Improved JWK handling to comply with the spec https://datatracker.ietf.org/doc/html/rfc7517. Implementation inspired by https://github.com/RustCrypto/JOSE.