-
Notifications
You must be signed in to change notification settings - Fork 135
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: implement didcomm v2 packing/unpacking #575
Conversation
67e250f
to
1e5b276
Compare
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.
i am not ready with the review yet but have already a few questions that need to be resolved before i would approve.
packages/did-comm/src/__tests__/did-comm.action-handler.test.ts
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## next #575 +/- ##
===========================================
+ Coverage 67.58% 77.77% +10.19%
===========================================
Files 62 72 +10
Lines 1530 2227 +697
Branches 247 396 +149
===========================================
+ Hits 1034 1732 +698
- Misses 400 493 +93
+ Partials 96 2 -94 |
5ca3d15
to
46aa1eb
Compare
packages/core/src/types/IResolver.ts
Outdated
@@ -59,4 +65,61 @@ export interface IResolver extends IPluginMethodMap { | |||
* @public | |||
*/ | |||
resolveDid(args: ResolveDidArgs): Promise<DIDResolutionResult> | |||
|
|||
/** | |||
* Dereferences a DID URI fragment and return the corresponding DID document entry. |
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.
* Dereferences a DID URI fragment and return the corresponding DID document entry. | |
* Dereferences a DID URL fragment and returns the corresponding DID document entry. |
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.
lgtm, it would be just nice to change the one argument of the getDIDCommMessageMediaType
Implement
IDIDComm.packDIDCommMessage()
andIDIDComm.unpackDIDCommMessage()
for plaintext, signed and encrypted (auth and anon) DIDComm.fixes #558
fixes #559
This also adds a
getDIDComponentById
method to theIResolver
plugin, which can be used to get aVerificationMethod
or aServiceEndpoint
object form a DID document using anid
.