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

Update JWK examples to match the BLS JWK draft #58

Closed
kdenhartog opened this issue May 31, 2021 · 5 comments
Closed

Update JWK examples to match the BLS JWK draft #58

kdenhartog opened this issue May 31, 2021 · 5 comments
Assignees

Comments

@kdenhartog
Copy link
Member

see here for documentation: https://github.com/mattrglobal/bls12381-jwk-draft/blob/main/draft-denhartog-pairing-curves-jose-cose.txt

Key take aways

  1. Uses OKP instead of EC
  2. Strings have been modified a bit based on discussion with @OR13 and @troyronda who were known implementors at the time.
@kdenhartog kdenhartog self-assigned this May 31, 2021
@OR13
Copy link
Contributor

OR13 commented Jun 1, 2021

@kdenhartog make sure to update this everywhere, as soon as the draft is stable...

especially in the security vocab.

@kdenhartog
Copy link
Member Author

kdenhartog commented Jun 7, 2021

How stable are you thinking here? Like RFC for informational draft and registered in IANA or "we think we've got it solved and are waiting for standards process to finish".

I've sent a message to secdir at IETF in hopes of finding a way to take this through the RFC process at IETF, but I've not heard back yet. IETF's process is still an enigma to me.

@OR13
Copy link
Contributor

OR13 commented Jun 7, 2021

@kdenhartog IMO, as soon as you have a perma url for a draft submitted to an SDO its worth updating the ccg document

@kdenhartog
Copy link
Member Author

kdenhartog commented Jun 9, 2021

Sweet, I've got it here now: https://datatracker.ietf.org/doc/draft-denhartog-pairing-curves-jose-cose/

I'm going to be taken away from community contributions for a week or two while I help out our engineering team, but will take care of this when I've got the time to update it.

@Wind4Greg
Copy link
Collaborator

Updated document (PR #99) does not use JWK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants