-
Notifications
You must be signed in to change notification settings - Fork 204
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
feat: add support for key type k256 #1722
feat: add support for key type k256 #1722
Conversation
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
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.
Very nice!!! :)
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
…o-ts into feat/add-key-k256
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Do the tests run locally for you @sairanjit? It keeps failing, and I'm not sure if it's related to your changes or not |
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Head branch was pushed to by a user without write access
Ah I merged #1629 and now there's some merge conflicts. Can you address it? |
I will check the tests and also run locally again |
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Signed-off-by: Sai Ranjit Tummalapalli <[email protected]>
Summary
k256
did:key
for the curve -secp256k1