Skip to content
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

Closed
wants to merge 11 commits into from
Closed

Conversation

hugoib
Copy link
Collaborator

@hugoib hugoib commented Oct 18, 2023

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.

@hugoib hugoib requested a review from IngridPuppet October 18, 2023 12:45
@hugoib hugoib changed the title Modified JWK to reflect spec Draft: Modified JWK to reflect spec Oct 18, 2023
@hugoib
Copy link
Collaborator Author

hugoib commented Oct 18, 2023

On hold, tests are broken inside did-endpoint, fixing...

@hugoib hugoib changed the title Draft: Modified JWK to reflect spec Modified JWK to reflect spec Oct 19, 2023
@hugoib
Copy link
Collaborator Author

hugoib commented Oct 19, 2023

Ready for review again 👍🏻

Copy link
Collaborator

@IngridPuppet IngridPuppet left a 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.

@hugoib
Copy link
Collaborator Author

hugoib commented Oct 23, 2023

@IngridPuppet I tracked down the problem. I am using https://github.com/jinxdash/prettier-plugin-rust inside my VS Code. I will disable it.

did-utils/src/key/bytes.rs Outdated Show resolved Hide resolved
did-utils/src/key/mod.rs Outdated Show resolved Hide resolved
did-utils/src/key/secret.rs Outdated Show resolved Hide resolved
did-endpoint/src/didgen.rs Outdated Show resolved Hide resolved
@francis-pouatcha francis-pouatcha requested review from francis-pouatcha and removed request for IngridPuppet October 24, 2023 17:29
@francis-pouatcha
Copy link
Member

Formating makes review tough.

Copy link
Member

@francis-pouatcha francis-pouatcha left a 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.

@hugoib
Copy link
Collaborator Author

hugoib commented Nov 8, 2023

Closed and opened a new, cleaner: #70

francis-pouatcha added a commit that referenced this pull request Nov 9, 2023
@hugoib hugoib deleted the iss-49--jwk-to-reflect-spec branch November 9, 2023 15:42
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.

Modify our JWK implementation in did-utils to reflect spec
3 participants