-
Notifications
You must be signed in to change notification settings - Fork 110
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
Dereferencing relative to issuer
#914
Comments
This should be consistent with standard URI "relative resolution" as explained in https://datatracker.ietf.org/doc/html/rfc3986#section-5.2. |
How is this related to #709 |
|
Related to the "controller" is "issuer" comment that Manu made regarding Data Integrity. https://github.com/digitalbazaar/vc-js/blob/main/lib/CredentialIssuancePurpose.js#L72 |
One option is to make it so that if you dereference the issuer value, you get back a "controller document" that has verification relationships and associated verification methods. The |
@dlongley your suggestion is the best I have heard so far... but... can we do better? |
The issue was discussed in a meeting on 2022-10-12
View the transcript6.1. Dereferencing relative to
|
We have to specify how this works in the vc-data-integrity spec. I'll link to that PR when we have it... should be in the next couple of weeks, hopefully. |
The issue was discussed in a meeting on 2023-01-11
View the transcript3.1. Dereferencing relative to
|
The issue was discussed in a meeting on 2023-02-14
View the transcript4. Extension Points.Brent Zundel: look at these things that need to be fixed.
Brent Zundel: we need to point to something.
Brent Zundel: the reason for this, is because... we have been dinged on our charter for this stuff.
Orie Steele: illuminates us with the wisdom of having 10k registries.... then ends snark and procedes to suggest that unless we can define one normative type for each item on this list that they should be dropped.
Orie Steele: removing them from the doc does not mean they aren't useful or fine, just that they don't belong in the spec.
Manu Sporny: there is also a render extension.
Manu Sporny: in many cases, its just a URL. Brent Zundel: Can we say more than, it is just a URL. Manu Sporny: what about a human readable web page?. Brent Zundel: we need at least 1 way to do things. Dave Longley: we should also say, these id's depending on where they occur, they don't need to derefernence to things.
Dave Longley: we should describe what a holder dererefernces too, it should go to a controller document, like issuer.
Michael Prorock: +1 to uuid, maybe we cover URN in URL examples as well.
Michael Prorock: should be integrity protect schemas, how?.
Michael Prorock: we should describe what we are seeing in the wild. Gabe Cohen: we have one in the CCG. Joe Andrieu: I am in favor of removing things that are not defined....
Joe Andrieu: URLs being here is because of WHATWG... confusion. Kristina Yasuda: the property part... status we have a work item. David Chadwick: Not sure we need to point to standards.
David Chadwick: does it have to be a standard? seems like the bar is too high. Dmitri Zagidulin: concrete proposal, we remove these sections from the spec, but then reserve the terms. Brent Zundel: our charter does say, for all normative required properties....
Brent Zundel: we acknowledged there is a problem here, but it does not mean that we should aim for the low bar we set..
Brent Zundel: lets consider why we are keeping things we can't provide references for.
Orie Steele: notes variations on VC JSON Schema and other ways of linking json schema to credentials and formats.
Orie Steele: maybe we don't have to point this to a normative standard.
Orie Steele: 100% agreement to remove all this stuff.
Orie Steele: +1 dimitriz. Brent Zundel: I understand we have the same notion regardingn Downref. Michael Jones: its a bit of a tangent, on WHATWG.
Joe Andrieu: I am a big fan of removing first, and adding good stuff that meets our bar back.
Joe Andrieu: I am opposed to "reserved terms", because it would prevent people from experimenting.
Joe Andrieu: the question I have is regarding termsOfService.
Joe Andrieu: doc searl is working on something related to that... maybe its an option. Manu Sporny: the objectors, we spoke to them... they wanted proof there is interop on these things.
Manu Sporny: we can just point to CCG things.
Manu Sporny: I think I am -1 to removing things pre-maturely.
Manu Sporny: we would just violate the spec, if we remove things we use in production.
Manu Sporny: render is simple enough... we should be able to do that in CR.
Manu Sporny: status list keep, credential schema lets point to the ccg, refresh service has an implementation and people using it, evidence is ok, termsOfUse seems poorly supported currently.
Manu Sporny: is anyone using termsOfUse?. David Chadwick: We used it in tests.
Brent Zundel: If there is something on this list, that you are using, will you raise a PR to modify the spec to point to something?.
Brent Zundel: slide 72.
Brent Zundel: we need to concretly move forward.
Michael Prorock: The danger of reserved, sounds great, but ... maybe its not a good solution.
Michael Prorock: it seems reasonable to clarify language on URLs. David Chadwick: I covered termOfUse. Paul Dietrich: was there any conclusion on dereference URLs vs non dereference?.
Brent Zundel: We seem to want to clarify URL vs URN as legal for use cases.
Michael Prorock: we can cover cases for both URL and URN, DID etc.... See github issue vc-data-model#914.
Joe Andrieu: We should back away from WHATWG.
Joe Andrieu: so seems like we should not follow that guidance.
Joe Andrieu: question for Manu, no snark implied.
Joe Andrieu: my understanding is that your current VCs won't break anything, because you already cover this use case under 1.1.
Manu Sporny: Do we want to open that can of worms.
Manu Sporny: why are we trying to do work that was not asked for by objectors.
Manu Sporny: the reason we added extension points, was to allow people to use them. Michael Prorock: we don't have requirements for this.
Manu Sporny: we don't need to describe anything, just let people use the extension points.
Manu Sporny: concern is that we are solving a non problem.
|
data integrity did a good job on this: https://w3c.github.io/vc-data-integrity/#retrieve-verification-method I suggest we close this, and note that we do not intend to address this in the core data model. |
agreed that this topic should be addressed in the securing specs (vc-jwt and data integrity) |
The issue was discussed in a meeting on 2023-04-11
View the transcript1.6. (issue vc-data-model#914)See github issue vc-data-model#914. Kristina Yasuda: "Dereferencing relative to issuer" - We should add guidance somewhere regarding dereferencing relative to an issuer.. Orie Steele: believes Data Integrity has solved for this and that VC-JWT has not.
Orie Steele: this is a fundamental issue - data integrity defining this in DI spec implies this issue should be closed in favor of solving in VC-JWT.
Brent Zundel: feels like this is open elsewhere and should be closed here. Kristina Yasuda: good. closed.. |
1 similar comment
The issue was discussed in a meeting on 2023-04-11
View the transcript1.6. (issue vc-data-model#914)See github issue vc-data-model#914. Kristina Yasuda: "Dereferencing relative to issuer" - We should add guidance somewhere regarding dereferencing relative to an issuer.. Orie Steele: believes Data Integrity has solved for this and that VC-JWT has not.
Orie Steele: this is a fundamental issue - data integrity defining this in DI spec implies this issue should be closed in favor of solving in VC-JWT.
Brent Zundel: feels like this is open elsewhere and should be closed here. Kristina Yasuda: good. closed.. |
We should add guidance somewhere regarding dereferencing relative to an issuer.
See also decentralized-identity/ion#285
For example:
issuer
+proof.verificationMethod
=>{ publicKeyJwk }
iss
+kid
=>{ publicKeyJwk }
The text was updated successfully, but these errors were encountered: