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 Cryptography key abstraction for signable, verifiable keys, and also private and public base impls #67

Merged
merged 7 commits into from
Aug 10, 2023

Conversation

elribonazo
Copy link
Contributor

@elribonazo elribonazo commented Jun 24, 2023

Description

Implemented basic cryptographic key abstraction, our plan is to introduce it now and extend this over time with more abstractions.

Current abstractions:

  1. Signable
  2. Verifiable
  3. PrivateKey

PrivateKeys have now the raw value always available without caring which key curve is used.
This abstraction has simplified our implementation of the agent too and we plan to extend it with more abstractions.

Exportable
Restorable
Derivable
Storable

Jira link

here

We have not changed the public API at all, so no updates in documentation are required.

Checklist

  • Self-reviewed the diff
  • New code has inline documentation
  • New code has proper comments/tests
  • Any changes not covered by tests have been tested manually

# Conflicts:
#	src/apollo/Apollo.ts
#	src/apollo/utils/Ed25519KeyCommon.ts
#	src/apollo/utils/Ed25519KeyPair.ts
#	src/apollo/utils/Ed25519PrivateKey.ts
#	src/apollo/utils/Ed25519PublicKey.ts
#	src/apollo/utils/Secp256k1KeyCommon.ts
#	src/apollo/utils/Secp256k1KeyPair.ts
#	src/apollo/utils/Secp256k1PrivateKey.ts
#	src/apollo/utils/Secp256k1PublicKey.ts
#	src/apollo/utils/X25519KeyCommon.ts
#	src/apollo/utils/X25519KeyPair.ts
#	src/apollo/utils/X25519PrivateKey.ts
#	src/apollo/utils/X25519PublicKey.ts
#	src/castor/Castor.ts
#	src/castor/did/prismDID/PrismDIDPublicKey.ts
@elribonazo elribonazo closed this Jun 24, 2023
@elribonazo elribonazo reopened this Jun 24, 2023
@elribonazo elribonazo closed this Jun 24, 2023
@elribonazo elribonazo reopened this Jun 24, 2023
@elribonazo elribonazo changed the title feat: Cryptographic key abstraction fix: Implement Cryptography key abstraction for signable, verifiable keys, including also (private and public base impls) Jun 27, 2023
@elribonazo elribonazo changed the title fix: Implement Cryptography key abstraction for signable, verifiable keys, including also (private and public base impls) fix: Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls Jun 27, 2023
@elribonazo elribonazo changed the title fix: Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls feat: Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls Jun 30, 2023
@elribonazo elribonazo merged commit 61d9dbd into master Aug 10, 2023
2 checks passed
@elribonazo elribonazo deleted the features/key-abstraction branch August 10, 2023 14:07
atala-dev added a commit that referenced this pull request Aug 17, 2023
### Bug Fixes

* update dates parsing from JWT credentials ([#66](#66)) ([24562f7](24562f7))

### Features

* Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls ([#67](#67)) ([61d9dbd](61d9dbd))
github-actions bot pushed a commit that referenced this pull request Aug 17, 2023
….3...v2.1.0) (2023-08-17)

### Bug Fixes

* update dates parsing from JWT credentials ([#66](#66)) ([24562f7](24562f7))

### Features

* Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls ([#67](#67)) ([61d9dbd](61d9dbd))
amagyar-iohk pushed a commit that referenced this pull request Aug 18, 2023
* update dates parsing from JWT credentials ([#66](#66)) ([24562f7](24562f7))

* Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls ([#67](#67)) ([61d9dbd](61d9dbd))
github-actions bot pushed a commit that referenced this pull request Aug 23, 2023
… (2023-08-17)

* update dates parsing from JWT credentials ([#66](#66)) ([24562f7](24562f7))

* Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls ([#67](#67)) ([61d9dbd](61d9dbd))
mixmix pushed a commit to mixmix/atala-prism-wallet-sdk-ts that referenced this pull request May 6, 2024
… and also private and public base impls (hyperledger#67)

Signed-off-by: Francisco Javier Ribó Labrador <elribonazo@gmail.com>
mixmix pushed a commit to mixmix/atala-prism-wallet-sdk-ts that referenced this pull request May 6, 2024
…2.0.3...v2.1.0) (2023-08-17)

* update dates parsing from JWT credentials ([hyperledger#66](hyperledger#66)) ([24562f7](hyperledger@24562f7))

* Add Cryptography key abstraction for signable, verifiable keys, and also private and public base impls ([hyperledger#67](hyperledger#67)) ([61d9dbd](hyperledger@61d9dbd))

Signed-off-by: Francisco Javier Ribó Labrador <elribonazo@gmail.com>
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.

2 participants