w3c / did-core

W3C Decentralized Identifier Specification v1.0
https://www.w3.org/TR/did-core/
Other
407 stars 95 forks source link

Controller confusion #819

Closed mwherman2000 closed 2 months ago

mwherman2000 commented 2 years ago

In https://www.w3.org/TR/did-core/#verification-method-properties, it says that controller is required. Clicking on controller in this table links to https://www.w3.org/TR/did-core/#did-controller ...which says controller is optional. These need to be differentiated or, at least, unlinked from one another.

kdenhartog commented 2 years ago

The controller property defined in https://www.w3.org/TR/did-core/#did-controller is for establishing the controller of the DID Document. The controller property defined in https://www.w3.org/TR/did-core/#verification-method-properties is about the controller of the verification method. They're two separate properties which follow different semantics because they're located in different objects of the JSON.

TallTed commented 2 years ago

The link from the controller in the Verification Method Properties table does need to change, from https://www.w3.org/TR/did-core/#dfn-controller (not from https://www.w3.org/TR/did-core/#did-controller as @mwherman2000 said, though did-controller is the fragment ID of the section header for the section which contains the dfn-controller fragment).

The current target of the link is within § 5.1.2 DID Controller

The correct target of the link is within § 5.2 Verification Methods -- but this definition does not now have a unique fragment identifier in the HTML.

msporny commented 2 months ago

PR #820 has been merged to address this issue.