Support resolution of "key" DID method #121
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New modules:
trustchain-ion/did_methods
DIDMethodsResult
.DIDMethod
s and implements theDIDResolver
trait to expose theresolve
method with a prior match on the method of the DID being resolved (eg. "ion" or "key" methods).DIDMethodsResult
is a wrapper forssi::did::DIDMethods
, re-implementing the fullDIDMethods
api in addition to aresolve_as_result
method, which is a Trustchain extension of theresolve
method (wrapping the returned tuple in aResult<T,ResolverError>
)Extent of
DIDMethodResult
usage:trustchain-ion/tests/resolver.rs
.IONResolver
s are used extensively in tests and library code, created withget_ion_resolver
, and none of these have been replaced with the more flexible DID Methods resolver (in many cases it doesn't make sense to support resolution of non-"ion" DIDs, eg. when resolving during ion verification inside theIONVerifier
).Changes with widespread effects:
get_ion_resolver(endpoint: &str)
refactored toget_ion_resolver(endpoint: &Endpoint)
, using theEndpoint
type from thetrustchain-ion