-
Notifications
You must be signed in to change notification settings - Fork 61
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
Resolve DIDDocument from cached events #426
Comments
Hi @nichonien, Sounds like a useful construct. A query would still need to be made to the underlying blockchain or verifiable data registry, to see if there is new information that would invalidate the cache, right? Is the idea that only the block height needs to be queried? It might also assist testing and particular applications to effectively enable constructing simple DID method implementations by pre-populating DID resolution results in the cache. Related issue about caching internally to VC verification: #263. Maybe lastBlock could be a DID resolution option (input metadata). It would just be a number, right? A non-blockchain generalization might be to use a timestamp. Providing the cached DID resolutions, and implementing the lastBlock option, could be done independently. |
Thanks @clehner for sharing the ongoing work and resources to understand better. Looks like the team has already envisioned possibility of
Yes, so the API would receive the
Agreed, also the hmm, the
I didn't really understand your reference here. |
Hi Team,
The
did
resolution toDIDDocument
takes blockchain interaction to execute and sometimes may not be efficient for large number of resolution calls. If theresolver
could have configurations / options to accept cacheddid
events and fetch the events beyond the cached scope to create the DIDDoc it would be much more faster I guess.So the resolver might look like,
resolver(did, cachedEvent[], lastBlock)
. ThelastBlock
param would be the block number till where the events have been cached.Let me know your thoughts. Thanks!
The text was updated successfully, but these errors were encountered: