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

Clarify when DID parameters should (not) be used. #144

Merged
merged 1 commit into from Jan 1, 2020

Conversation

peacekeeper
Copy link
Contributor

@peacekeeper peacekeeper commented Dec 17, 2019

In order to further discuss DID parameters (aka matrix parameters), it has been requested that we add text to clarify when to use them and when not to use them.


Preview | Diff

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.

Per the discussion in #137, I believe that this PR should be updated to clarify that matrix parameters MUST NOT be used. And that rather, query parameters and/or fragments should be used for the use cases that matrix parameters were previously being considered for.

@peacekeeper
Copy link
Contributor Author

peacekeeper commented Dec 17, 2019

@selfissued

this PR should be updated to clarify that matrix parameters MUST NOT be used.

This PR is simply clarifying when matrix parameters should be used and when they should not be used. It improves the current spec text which includes matrix parameters.

It's completely valid for the WG to propose to remove matrix parameters from the spec, but that should be in a different PR, and any such PR should also explain how the current use cases involving matrix parameter can still be fulfilled without them (or it should be argued why those use cases should not be supported anymore).

@peacekeeper
Copy link
Contributor Author

Merging, since this has been requested several times, I believe it's an improvement to the current text, and there have been no further comments in this thread.

This PR only adds clarification text about DID parameters. Any discussion or proposal to remove them from the spec should happen elsewhere.

@peacekeeper peacekeeper merged commit 601e0d9 into master Jan 1, 2020
@msporny msporny deleted the peacekeeper-clarify-matrix-parameters branch January 7, 2020 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants