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

Add "service-type" DID URL matrix parameter. #62

Closed
wants to merge 2 commits into from

Conversation

peacekeeper
Copy link
Contributor

@peacekeeper peacekeeper commented Oct 5, 2019

Re-creating PR from CCG repo: w3c-ccg/did-spec#191. Please consider earlier discussions there.


Preview | Diff

@peacekeeper
Copy link
Contributor Author

This adds one concrete DID URL matrix parameter.

Description: Identifies a set of services from the DID Document by service type.

Example: did:example:1234;service-type=agent

@dlongley
Copy link
Contributor

dlongley commented Oct 5, 2019

The purpose/use case for this parameter needs to be expressed. It's unclear why referencing a set via a URL would be more useful than simply passing a plain DID to an application and allowing it to make some kind of (more) informed selection of an appropriate service after dereferencing the DID Document.

@msporny
Copy link
Member

msporny commented Oct 10, 2019

Example: did:example:1234;service-type=agent

Same questions as in #60 (comment) ... and I'd expect that question to be true of any type-based matrix parameter.

Copy link
Contributor

@selfissued selfissued left a comment

Choose a reason for hiding this comment

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

I believe that we should not merge this until we've determined whether there is actually consensus to resurrect the matrix parameter syntax at all. Less is more.

@peacekeeper
Copy link
Contributor Author

Regarding a potential service-type parameter, it may be interesting to note that there is a media type text/uri-list (RFC2483). In other words, it wouldn't be completely unreasonable to argue that dereferencing a URI could result in a list of other URLs.

@OR13
Copy link
Contributor

OR13 commented Feb 7, 2020

@peacekeeper any update on this after the F2F?

@msporny
Copy link
Member

msporny commented Feb 7, 2020

@peacekeeper any update on this after the F2F?

@peacekeeper is working with @jandrieu to create use cases for matrix parameters. The use cases for matrix parameters weren't clear (as in, not clearly articulated and there were a number of people that were arguing that the matrix parameter use cases could be solved via other means, like resolver parameters).

Next step is to get use cases for matrix params, which have been renamed to "DID Parameters", and then see if we need special syntax for that or if we can just use query parameters.

@OR13
Copy link
Contributor

OR13 commented Mar 15, 2020

This PR is stale, has conflicts and is blocked pending discussion of DID Parameters. I'd prefer to close it for now, to focus the attention of the WG on PRs that are not blocked. @peacekeeper thoughts?

@kdenhartog kdenhartog added the pending close Issue will be closed shortly if no objections label Apr 7, 2020
@kdenhartog
Copy link
Member

kdenhartog commented Apr 7, 2020

Marked pending close after the resolution 1 on the 4/7/2020 call.

@peacekeeper
Copy link
Contributor Author

Marked pending close after the resolution 1 on the 4/7/2020 call.

I agree this should be closed, but the reason for closing is that there has been consensus that there is no need for having service-type as a DID parameter in the DID URL. The resolution on the 2020-04-07 call was about the syntax of DID parameters, which is independent of the question which DID parameters we want.

@msporny
Copy link
Member

msporny commented Apr 10, 2020

Let's take this up on the next special topic call since we're still discussing the general area of DID Parameters.

@msporny msporny closed this Apr 20, 2020
@msporny msporny deleted the peacekeeper-matrix-parameter-service-type branch May 1, 2020 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending close Issue will be closed shortly if no objections
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants