Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Add "key" DID URL matrix parameter. #192

Closed
wants to merge 1 commit into from

Conversation

peacekeeper
Copy link
Member

@peacekeeper peacekeeper commented Apr 27, 2019

This adds one concrete DID URL matrix parameter. See #189.

Description: Identifies a key from the DID Document by key id.

Example: did:example:1234;key=mykey


Preview | Diff

@peacekeeper
Copy link
Member Author

Note that @dlongley has pointed out in Proposal for Revisions to Decentralized Identifiers Section of DID Spec that this matrix parameter may be unnecessary, since it is always possible to reference any resource inside a DID Document using fragment DID URL syntax. For example, did:example:1234;key=mykey may be equivalent to did:example:1234#mykey.

@@ -825,6 +825,15 @@ <h2>
</thead>

<tbody>
<tr>
<td>
<code>key</code>
Copy link
Contributor

@msporny msporny May 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keys already have ids... instead of this:

did:example:1234;key=mykey

Why can't you just do this?

did:example:1234#mykey

@peacekeeper peacekeeper force-pushed the peacekeeper-matrix-parameter-key branch from 0efd254 to 9d43d8b Compare May 10, 2019 11:40
@peacekeeper peacekeeper changed the base branch from peacekeeper-matrix-parameters-new to gh-pages May 10, 2019 11:41
@rhiaro rhiaro added the discuss Wider discussion in an issue or meeting required before merging label Aug 1, 2019
@brentzundel
Copy link
Contributor

This repo is scheduled to be archived. The work has moved to the DID WG.
The artifacts in the DID WG repository share commit history with this repo, so it should be possible to raise this PR against that repo.

@peacekeeper
Copy link
Member Author

Closing. This PR has been re-created in the DIDWG repo: w3c/did#59

@peacekeeper peacekeeper closed this Oct 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discuss Wider discussion in an issue or meeting required before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants