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

Decoupling DID Core spec from LD-Proof / LDS specs #154

Closed
awoie opened this issue Jan 22, 2020 · 18 comments
Closed

Decoupling DID Core spec from LD-Proof / LDS specs #154

awoie opened this issue Jan 22, 2020 · 18 comments
Assignees
Labels
extensibility related to extensibility, json-ld contexts, external properties, etc pending close Issue will be closed shortly if no objections

Comments

@awoie
Copy link
Contributor

awoie commented Jan 22, 2020

We should decouple the DID Core spec from LD-Proofs to allow people to use DIDs without understanding LD-Proofs.

For example ...

  • proof purposes are specified in the LD-Proof specification such as assertionMethod, authentication etc. We should always keep in mind that people want to use keys in DID Docs to generate proofs other than LD-Proofs as well, e.g., signing pdfs (probably not the best example). It feels a bit weird that someone needs to look at the LD-Proof spec to understand the different proof purposes.
  • allow other proofs for DID Documents, e.g., JWS.
@OR13
Copy link
Contributor

OR13 commented Jan 22, 2020

I think this is actually a request to have the jsonld context documented, since that is what is required for both VC JWT and VC Linked Data Proofs.

@OR13
Copy link
Contributor

OR13 commented Jan 22, 2020

Or rather, its not possible to use DID Docs for anything unless we understand what the properties mean, and in order to do that, we need to document every property in the existing jsonld context, as it applies to DIDs and VCs... and if we remove the context, then we would need to do the same thing with something else, like json schema, and ensure that they aligned or we would have a massively broken VC ecosystem...

@awoie
Copy link
Contributor Author

awoie commented Jan 22, 2020

@OR13 Thanks for the additional comments. That is what the intention was.

@OR13
Copy link
Contributor

OR13 commented Jan 23, 2020

Related issues... in sidetree: decentralized-identity/sidetree#362

@OR13
Copy link
Contributor

OR13 commented Jan 25, 2020

Also related:

IMO the did core context should not rely on "sec": "https://w3id.org/security#",... since we cannot ensure that it is updated in a timely manner.

We might want to import the security context and its definitions into the did core spec... So that documentation can be fixed.

@rhiaro rhiaro added the extensibility related to extensibility, json-ld contexts, external properties, etc label Feb 4, 2020
@OR13
Copy link
Contributor

OR13 commented Mar 17, 2020

Blocked by registries discussion

please review:

w3c/did-spec-registries#14

w3c/did-spec-registries#11

@iherman
Copy link
Member

iherman commented Mar 17, 2020

See also discussions on https://www.w3.org/2020/03/16-did-minutes.html

@jonnycrunch
Copy link
Contributor

This is also blocked as LD-Proofs are dependent on the RDF normalization algorithm which is draft and at least I am having inconsistent results.

@OR13
Copy link
Contributor

OR13 commented Mar 17, 2020

@jonnycrunch
Copy link
Contributor

Thanks. I mostly use Go and Rust. 9 days ago https://github.com/piprate/json-gold (Go) was updated to use JSON-LD 1.1. so, I still need to do some interop testing.

@msporny
Copy link
Member

msporny commented Jun 2, 2020

The direction that we're heading in here is decoupling the DID Core spec from the Linked Data Security specs. We are in the process of removing proof, created, updated, and a variety of other Linked Data Security-y things.

@burnburn burnburn assigned OR13 and unassigned awoie Jun 2, 2020
@OR13
Copy link
Contributor

OR13 commented Jun 2, 2020

I think this has been address by w3c/did-spec-registries#47

@awoie
Copy link
Contributor Author

awoie commented Jun 2, 2020

I agree that this addresses my initial issue.

@jonnycrunch
Copy link
Contributor

section 7.9 Proof states that the proof property, the value MUST be a valid JSON-LD proof. I am having difficulty conforming to the spec with the CBOR representation with this constraint. I will make a PR to update this section to address my concerns.

@OR13
Copy link
Contributor

OR13 commented Jun 2, 2020

PRs to remove LD Proof stuff from core / registries welcome.

@msporny
Copy link
Member

msporny commented Jun 16, 2020

All normative and informative references have been removed from the specification, we can close this issue.

@jonnycrunch
Copy link
Contributor

will do

@msporny msporny added the pending close Issue will be closed shortly if no objections label Jun 16, 2020
@brentzundel
Copy link
Member

No comments since marked pending close, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extensibility related to extensibility, json-ld contexts, external properties, etc pending close Issue will be closed shortly if no objections
Projects
None yet
Development

No branches or pull requests

7 participants