Skip to content
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

5.3.1 Semantic Interoperability #525

Closed
nadalin opened this issue Apr 3, 2019 · 6 comments
Closed

5.3.1 Semantic Interoperability #525

nadalin opened this issue Apr 3, 2019 · 6 comments
Labels
pending close Close if no objection within 7 days
Milestone

Comments

@nadalin
Copy link

nadalin commented Apr 3, 2019

This is a confusing section as there is no motivation for this section as most JSON processors don't necessary parse and understand the @context yet this section says that they MUST, I don't see any rational for this, can someone explain?

Current JWT parsers don't know what to do with an @context and thus would ignore

@David-Chadwick
Copy link
Contributor

If you think a bit deeper about this, your message indicates precisely why @context or something similar is needed - JSON processors have no idea how to support any property that they have not already been told about, so have no support for extensibility except by out of band means.

@nadalin
Copy link
Author

nadalin commented Apr 3, 2019

@David-Chadwick sorry that does not process for me, as i'm talking about JWT processors

@burnburn burnburn added this to the CR-Exit milestone Apr 4, 2019
@msporny
Copy link
Member

msporny commented Apr 5, 2019

@nadalin wrote:

This is a confusing section as there is no motivation for this section as most JSON processors don't necessary parse and understand the @context yet this section says that they MUST

The intent of that conformance statement is to state that Verifiable Credential processors that are operating in JSON mode MUST ensure that the array of values associated with @context are in the expected order for the particular use case (and that's it). It specifically does not mean that they must dereference @context URLs or anything of the sort.

This rule is there to ensure that JSON-based Verifiable Credential processors do not need to use any JSON-LD libraries to be conformant with the specification AND ensure semantic interoperability w/ Verifiable Credentials processors that DO process in JSON-LD mode.

We can suggest that this normative statement is elaborated upon so that it's easier to understand the reasoning behind the requirement for implementers.

@stonematt
Copy link
Contributor

stonematt commented Apr 16, 2019

Resolved on VCWG call 4/16/2019.
RESOLVED: Editors will non-normatively elaborate on the rules that ensure that the array of values associated with @context should be in the expected order. Close issue #525 after the PR has been merged.

Plan to close issue 7days

@stonematt stonematt added the pending close Close if no objection within 7 days label Apr 16, 2019
@nadalin
Copy link
Author

nadalin commented Apr 16, 2019

@stonematt Will wait to see final wording as I'm not convinced this will be adequate

@burnburn
Copy link
Contributor

The outstanding concern by @nadalin regarding JSON-LD will be tracked by issue #633. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending close Close if no objection within 7 days
Projects
None yet
Development

No branches or pull requests

5 participants