IDunion / indy-did-resolver

Resolver for the did:indy method
Apache License 2.0
8 stars 7 forks source link

Build RevRegDelta request from RevRegEntry path with appropriate quer… #19

Closed domwoe closed 2 years ago

domwoe commented 2 years ago

…y parameters

Resolves #13

I've kept the RevRegDelta path for now as well.

Signed-off-by: Dominic Wörner dom.woe@gmail.com

c2bo commented 2 years ago

Finally got the time to test this, sorry for the delay :/ Looks good to me - my only question would be if we want to serve the ledger objects directly or also some form of metadata (e.g. stateproof) similar to how the did document is served.

domwoe commented 2 years ago

This should be the case.

Similar to didDocumentMetadata with a node_response field there's contentMetadata with a node_response field.

c2bo commented 2 years ago

I had to merge main into the branch and everything worked directly with contentMetadata, nice!