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

Initial DIDComm v2 envelope support via Askar backend #1331

Merged
merged 6 commits into from
Oct 21, 2021

Conversation

andrewwhitehead
Copy link
Contributor

Also cleans up and relocates the (incomplete) in-memory ECDH-1PU support.

This does not make the DIDComm v2 envelope format(s) available for communication yet, it only establishes some of the basic functionality.

@swcurran
Copy link
Contributor

What about adding a "DIDComm2Envelopes.md" that talks about how to use this? How do we get this tested?

@ianco @shaangill025 -- can you please take a look at the code, at least?

Thanks! This is another very cool addition.

@swcurran
Copy link
Contributor

Also -- is there any feedback to add to RFC 587 after the implementation? I know you are commenting in the DIDComm2 Working Group, but wanted to make sure what's needed is getting back to the Aries community. Thanks!

@andrewwhitehead
Copy link
Contributor Author

@swcurran There isn't really a way to use it at the moment, at least for agent-to-agent communication. We still need to implement a standard key lookup interface for the agent's own DID document keys as well as other agents' keys via the DID resolver (and cache). This is mainly a proof of concept to see what the complexity is and what it left to do.

@swcurran
Copy link
Contributor

OK -- perhaps @troyronda and @Baha-sk might be interested in taking a look?

@codecov-commenter
Copy link

codecov-commenter commented Jul 23, 2021

Codecov Report

Merging #1331 (61ee23c) into main (0c266fa) will increase coverage by 0.07%.
The diff coverage is 92.52%.

@@            Coverage Diff             @@
##             main    #1331      +/-   ##
==========================================
+ Coverage   95.75%   95.82%   +0.07%     
==========================================
  Files         485      488       +3     
  Lines       29779    30009     +230     
==========================================
+ Hits        28515    28757     +242     
+ Misses       1264     1252      -12     

…tes to JWE handling

Signed-off-by: Andrew Whitehead <cywolf@gmail.com>
… and variables for consistency

Signed-off-by: Andrew Whitehead <cywolf@gmail.com>
Signed-off-by: Andrew Whitehead <cywolf@gmail.com>
Signed-off-by: Andrew Whitehead <cywolf@gmail.com>
@swcurran
Copy link
Contributor

swcurran commented Oct 6, 2021

@andrewwhitehead -- can you please update. @ianco, once done, can you please review? Thanks!

Copy link
Contributor

@ianco ianco left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ianco ianco merged commit 23b8aee into openwallet-foundation:main Oct 21, 2021
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.

5 participants