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

Update DID Method Registry to clarify it's not an endorsement #519

Merged
merged 2 commits into from
Jun 26, 2023

Conversation

kdenhartog
Copy link
Member

@kdenhartog kdenhartog commented Jun 20, 2023

This is necessary because some people believe that listing a method within this registry is somehow an endorsement of the method and it's underlying technology. This clarifies this was not the case, nor was it the goal of the Working group.


Preview | Diff

This is necessary because some people believe that listing a method within this registry is somehow an endorsement of the method and it's underlying technology. This clarifies this was not the case, nor was it the goal of the Working group.
@kdenhartog
Copy link
Member Author

cc @melvincarvalho

Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

Approved with suggested changes.

Given that this is an Editorial change that is aligned with the intention of the DID Spec Registries, as decided during the DID WG's operation, we can merge this at any point.

CC'ing W3C DID WG Chairs to get explicit approval before merging: /cc @brentzundel @burnburn

index.html Outdated Show resolved Hide resolved
Copy link
Contributor

@mprorock mprorock left a comment

Choose a reason for hiding this comment

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

Would like to see adjustments from @msporny merged and feedback in from chairs / W3C staff

Generally a fan, as I think this clarifies that this is just a registry of specifications

Co-authored-by: Manu Sporny <msporny@digitalbazaar.com>
Copy link
Contributor

@peacekeeper peacekeeper left a comment

Choose a reason for hiding this comment

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

Makes sense.

Copy link
Member

@brentzundel brentzundel left a comment

Choose a reason for hiding this comment

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

Thank you for raising this @kdenhartog

@brentzundel
Copy link
Member

@mprorock the requested changes have been made, and the chairs have conferred and approve. Could you re-review?

@shigeya
Copy link

shigeya commented Jun 21, 2023

Thanks for raising this PR (I came from the email thread). The added language is clear.

Copy link

@burnburn burnburn left a comment

Choose a reason for hiding this comment

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

Not sure why my approval from 2 days ago didn't stick, but I 100% agree with this PR (with Manu's changes applied).

@mprorock
Copy link
Contributor

@mprorock the requested changes have been made, and the chairs have conferred and approve. Could you re-review?

Approved

@msporny
Copy link
Member

msporny commented Jun 22, 2023

There are more than enough approvals/reviews for this to go in. I'm going to wait the requisite 7 days to see if there are any objections, and if not, merge the PR at that point.

@mccown
Copy link

mccown commented Jun 22, 2023

This is a great proposal -- to explicitly state that the DID Registry is only a list of methods that are compatible with the spec / technology.

After clarifying that a listing in the registry is not an endorsement, should we look at how entries get on the list? Specifically, should entries be self-asserted, manually verified, or have an automated compatibility process?

@ChristopherA
Copy link

+1 to this proposal.

@mccown
Copy link

mccown commented Jun 22, 2023

Before merging, I have another question. In the section "Status of This Document", it contains this sentence:

"This Group Note is endorsed by the Decentralized Identifier Working Group, but is not endorsed by W3C itself nor its Members."

Since the doc says "is endorsed" and later will say "does not act as an endorsement", does that create confusion? I'm wondering if the W3C legal dept may want to review the specific wording.

@kdenhartog
Copy link
Member Author

Ahh great catch @mccown! I think a better alternative would to say it's "recognized" or "managed" by the DID WG. This language could very well be creating some confusion here. I can add this change to the PR as well when I'm back at my computer.

@kdenhartog
Copy link
Member Author

This is a great proposal -- to explicitly state that the DID Registry is only a list of methods that are compatible with the spec / technology.

After clarifying that a listing in the registry is not an endorsement, should we look at how entries get on the list? Specifically, should entries be self-asserted, manually verified, or have an automated compatibility process?

I believe this is necessary as well. While I'm not a fan of utilizing a specific jurisdiction to censor the registry I absolutely believe we should be raising the bar on the requirements to be listed within the registry. I opened #433 to highlight this point because when I was an editor of this note, I noticed that I was essentially rubber-stamping methods and having to merge. I don't think the current state is an ideal situation either, so I'm all for revisiting this once a new WG is formed. Without a WG we're stuck in an interim state where large changes have to involve W3C management which would be a drain on their time. Let's discuss further in #433 and can take further action once a resolution is reached on the WG charter.

@brentzundel
Copy link
Member

The group note "DID Specification Registries" is endorsed by the DID WG. We published it. We believe that it is a good and useful thing.
In endorsing the Group Note, we endorse the policies according to which an entry can be added to the registry. Those policies explicitly do not require endorsement from the DID WG to add entries to the registry.

This means that the inclusion of a DID Method in the registry does not imply that the DID Method is endorsed by the DID WG, any more than a new DID Document property listed in the registry is endorsed by the DID WG.

@kdenhartog
Copy link
Member Author

That's a great perspective @brentzundel. When I read that it made me realize the nuance here and leads me to believe this may not require any further text changes.

@brentzundel brentzundel merged commit 4897706 into main Jun 26, 2023
1 check passed
@brentzundel
Copy link
Member

Multiple positive reviews, questions asked and answered, no further requests for changes or clarifications, merging.

@msporny msporny deleted the kdh/non-endorsement branch August 30, 2024 15:39
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