Skip to content

Conversation

@mickenordin
Copy link
Member

Fixes: #313

Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

This is already good, do you think we could provide an example of how to use the jwks.json? Also, at this point I'm ok to keep the deprecated part, but I'd remove the now-fully-outdated example.

@mickenordin
Copy link
Member Author

This is already good, do you think we could provide an example of how to use the jwks.json? Also, at this point I'm ok to keep the deprecated part, but I'd remove the now-fully-outdated example.

I removed the deprecated example now. Where would be a good place to put an example, and how would you like it to be? An example json with the various fields?

@glpatcern
Copy link
Member

I removed the deprecated example now. Where would be a good place to put an example, and how would you like it to be? An example json with the various fields?

Good question: what about an Appendix as it was done with the previous example? Then yes I'd include an example JSON like the one in #313, and a code snippet from your own toy implementation (it's python IIRC right?) to 1) encode and add a signature at the sender and 2) validate it at the receiver.

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.

Adopt JWS format for exposing public keys

3 participants