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

Add information about key agreement #53

Closed
2byrds opened this issue Sep 29, 2023 · 4 comments
Closed

Add information about key agreement #53

2byrds opened this issue Sep 29, 2023 · 4 comments

Comments

@2byrds
Copy link
Contributor

2byrds commented Sep 29, 2023

Should a Key Agreement key example be included?

@peacekeeper
Copy link
Contributor

I don't know how key agreement works in KERI, can you point me to something?

I know that Ed25519 keys can be converted to X25519 for key agreement, and we could add such converted keys to the DID document. Some DID methods do that "automatically", and some don't. There are some use cases and ecosystems where this is not acceptable.

@swcurran
Copy link
Contributor

There definitely needs to be a way to declare both the Verification and KeyAgreement, and have the controller explicitly do the derivation of (for example) Ed25519 to X25519 vs. the DID Method doing it for you.

@2byrds
Copy link
Contributor Author

2byrds commented Oct 3, 2023

Sam and Phil provided information on key agreement:
KERI supports more than Ed25519 (ECDSA for instance would use the same curve, no translation). So specifying translation is problematic, NIST algorithms use the same keys as the signing keys. The right strategy would be similar to how we are handling endpoints, domain migration info, etc. Normally in KERI we would use BADA-RUN for key agreement (just like we do for endpoints). But we should support two choices; BADA-RUN the assertion for the key agreement encryption key or anchor the key agreement info to the KEL via an ACDC credential.

@2byrds
Copy link
Contributor Author

2byrds commented Oct 5, 2023

I included this wording in #64
Happy to adjust in a future PR.

@2byrds 2byrds closed this as completed Oct 5, 2023
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

No branches or pull requests

3 participants