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

feat: add support for key type k256 #1722

Merged
merged 13 commits into from
Jan 31, 2024

Conversation

sairanjit
Copy link
Contributor

Summary

  • Added support for new KeyType - k256
  • Added support for did:key for the curve - secp256k1

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Copy link
Contributor

@TimoGlastra TimoGlastra left a comment

Choose a reason for hiding this comment

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

Very nice!!! :)

packages/core/src/crypto/jose/jwa/crv.ts Outdated Show resolved Hide resolved
packages/core/src/crypto/jose/jwk/K256Jwk.ts Outdated Show resolved Hide resolved
packages/core/src/crypto/jose/jwk/ecCompression.ts Outdated Show resolved Hide resolved
packages/core/src/crypto/jose/jwk/ecCompression.ts Outdated Show resolved Hide resolved
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra TimoGlastra enabled auto-merge (squash) January 31, 2024 13:10
@TimoGlastra
Copy link
Contributor

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 <sairanjit.tummalapalli@ayanworks.com>
auto-merge was automatically disabled January 31, 2024 15:19

Head branch was pushed to by a user without write access

@TimoGlastra
Copy link
Contributor

Ah I merged #1629 and now there's some merge conflicts. Can you address it?

@sairanjit
Copy link
Contributor Author

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

I will check the tests and also run locally again

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra TimoGlastra merged commit 22d5bff into openwallet-foundation:main Jan 31, 2024
7 checks passed
@sairanjit sairanjit deleted the feat/add-key-k256 branch February 1, 2024 05:54
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.

None yet

2 participants