-
Notifications
You must be signed in to change notification settings - Fork 90
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
Linked Verifiable Presentations #1398
Conversation
Hi @daniel-mader, thanks for submitting this. @eike-hass told me you had some concerns regarding the resolution of As we have done in other instances, we won't provide direct support for resolving For the latter matter, we will take care of the bindings as well as making sure CI won't complain about the code before merging it. |
Hey @UMR1352, thanks for your feedback! Regarding the resolution, my questions revolve around whether we should check if the link is valid at all or if we should completely treat it as some arbitrary URL and not even check if the resource can be fetched or not. Shall I also guide the user how to create a VP (without nonce) to be hosted or rather add a reference to the existing example? |
Checking if the URL are fetchable would require resolution of the resource. I think we could simply stick to checking for its syntactic validity. |
5d454c8
to
e220db2
Compare
bindings/wasm/src/credential/linked_verifiable_presentation_service.rs
Outdated
Show resolved
Hide resolved
…rvice.rs Co-authored-by: wulfraem <[email protected]>
Description of change
Adds support for Linked Verifiable Presentations following this DIF specification.
Implementation follows the existing
linked_domain_service.rs
.Links to any relevant issues
n/a
Type of change
Add an
x
to the boxes that are relevant to your changes.How the change has been tested
Change checklist
Add an
x
to the boxes that are relevant to your changes.