mark did:webplus as DIF recommended#77
mark did:webplus as DIF recommended#77jrayback wants to merge 4 commits intodecentralized-identity:mainfrom
Conversation
Signed-off-by: Jonathan Rayback <jrayback@gmail.com>
|
This PR indicates the beginning of the formal review period for |
ottomorac
left a comment
There was a problem hiding this comment.
Hello.
Whilst I believe the did method satisfies most requirements, we should get the W3C DID test suite sorted out first before approval.
I am following up with W3C folks to get this other PR from Victor approved.
|
Thanks, Otto, I appreciate that! |
|
@vdods - is there any update with respect to extending did:webplus support to SHA3? Are you planning to present an update to the group any time soon? |
|
Hi @Hkdolts -- progress is underway for the choice of hash function for each place a hash function is used, and SHA3 will be included. I'll post updates here, as well as mention them in the DIF WG call. |
|
@ottomorac I think you mean did:webplus :) |
dif-recommended/README.md
Outdated
| | `did:scid` | [Working Draft 03](https://lf-toip.atlassian.net/wiki/spaces/HOME/pages/88572360/DID+SCID+Method+Specification) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-scid.md) | No | Yes | No | ? | ? | ToIP-Approved at ToIP DID SCID Task Force | 0 | **No** | | ||
| | `did:web` | [Unofficial Draft](https://w3c-ccg.github.io/did-method-web/) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-web.md) | [Yes](https://github.com/w3c/did-test-suite/blob/main/packages/did-core-test-server/suites/implementations/did-web-spruce.json) | Yes | Yes | ? | ? | W3C Recommendation at W3C DID Methods WG | 0 | **No** | | ||
| | `did:webplus` | [Specification](https://ledgerdomain.github.io/did-webplus-spec/) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-webplus.md) | [Pending](https://github.com/w3c/did-test-suite/pull/228) | [In Progress](https://github.com/LedgerDomain/did-webplus) | Yes | [1](https://github.com/LedgerDomain/did-webplus) | In Progress | [OCI](https://oc-i.org) | [Intro](https://us02web.zoom.us/rec/share/1jJ7EIXc72ARQTQLXK6PXV2-cq0oN7gWlDBXwzD-IkEXpRfnxyxpX2IBZ1rRU23U.saBYCUDcfR2tbKps), [Deep Dive 1](https://us02web.zoom.us/rec/share/aF-Oyy6vsSHTQVotgcMMpdxAMo_I0e3PyvFHl5Wrqy3PbLMsl283eXGb2OBGV0Dr.-f9s4l5thUU_4JpS), [Deep Dive 2](https://us02web.zoom.us/rec/share/6yRpfB0ZND2JdmmYr6oQz8kYfFturosnG5ohQKxLNS4UXy80VyLuerzeNppo2XQ-.E74fBnNUdLYo1HFM) | **No** | | ||
| | `did:webplus` | [Specification](https://ledgerdomain.github.io/did-webplus-spec/) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-webplus.md) | [Pending](https://github.com/w3c/did-test-suite/pull/228) | [In Progress](https://github.com/LedgerDomain/did-webplus) | Yes | [1](https://github.com/LedgerDomain/did-webplus) | In Progress | [OCI](https://oc-i.org) | [Intro](https://us02web.zoom.us/rec/share/1jJ7EIXc72ARQTQLXK6PXV2-cq0oN7gWlDBXwzD-IkEXpRfnxyxpX2IBZ1rRU23U.saBYCUDcfR2tbKps), [Deep Dive 1](https://us02web.zoom.us/rec/share/aF-Oyy6vsSHTQVotgcMMpdxAMo_I0e3PyvFHl5Wrqy3PbLMsl283eXGb2OBGV0Dr.-f9s4l5thUU_4JpS), [Deep Dive 2](https://us02web.zoom.us/rec/share/6yRpfB0ZND2JdmmYr6oQz8kYfFturosnG5ohQKxLNS4UXy80VyLuerzeNppo2XQ-.E74fBnNUdLYo1HFM) | **Yes** | |
There was a problem hiding this comment.
| | `did:webplus` | [Specification](https://ledgerdomain.github.io/did-webplus-spec/) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-webplus.md) | [Pending](https://github.com/w3c/did-test-suite/pull/228) | [In Progress](https://github.com/LedgerDomain/did-webplus) | Yes | [1](https://github.com/LedgerDomain/did-webplus) | In Progress | [OCI](https://oc-i.org) | [Intro](https://us02web.zoom.us/rec/share/1jJ7EIXc72ARQTQLXK6PXV2-cq0oN7gWlDBXwzD-IkEXpRfnxyxpX2IBZ1rRU23U.saBYCUDcfR2tbKps), [Deep Dive 1](https://us02web.zoom.us/rec/share/aF-Oyy6vsSHTQVotgcMMpdxAMo_I0e3PyvFHl5Wrqy3PbLMsl283eXGb2OBGV0Dr.-f9s4l5thUU_4JpS), [Deep Dive 2](https://us02web.zoom.us/rec/share/6yRpfB0ZND2JdmmYr6oQz8kYfFturosnG5ohQKxLNS4UXy80VyLuerzeNppo2XQ-.E74fBnNUdLYo1HFM) | **Yes** | | |
| | `did:webplus` | [Specification](https://ledgerdomain.github.io/did-webplus-spec/) | Yes | [Yes](https://github.com/decentralized-identity/did-methods/blob/main/method-proposals/PROPOSAL-did-webplus.md) | [Yes](https://github.com/w3c/did-test-suite/pull/228) | [In Progress](https://github.com/LedgerDomain/did-webplus/blob/c7ec8c5591124961f9f2c90d43aab66d72e2454e/did-webplus/urd/src/listen.rs#L6) | Yes | [1](https://github.com/LedgerDomain/did-webplus) | In Progress | [OCI](https://oc-i.org) | [Intro](https://us02web.zoom.us/rec/share/1jJ7EIXc72ARQTQLXK6PXV2-cq0oN7gWlDBXwzD-IkEXpRfnxyxpX2IBZ1rRU23U.saBYCUDcfR2tbKps), [Deep Dive 1](https://us02web.zoom.us/rec/share/aF-Oyy6vsSHTQVotgcMMpdxAMo_I0e3PyvFHl5Wrqy3PbLMsl283eXGb2OBGV0Dr.-f9s4l5thUU_4JpS), [Deep Dive 2](https://us02web.zoom.us/rec/share/6yRpfB0ZND2JdmmYr6oQz8kYfFturosnG5ohQKxLNS4UXy80VyLuerzeNppo2XQ-.E74fBnNUdLYo1HFM) | **Yes** | |
There was a problem hiding this comment.
- Thanks to Otto, that
Pendingis now aYes😎 - the "in progress" is a little unconvincing on the Uniresolver driver so I linked directly into the relevant API in the current-commit code, for posterity 😅
…did-web-plus-as-recommended Signed-off-by: Jonathan Rayback <jrayback@gmail.com>
|
I think I would prefer to not change the order of entries in the list, i.e. not move the recommended ones to the top of the list. If we leave the order of entries as-is, we can better preserves the history of which method started the process in which order, and it also helps to avoid merge conflicts. But it's just a suggestion, no strong objection if others want to change the order. |
It was decided in the working group meeting of 21 Jan 26 to reformat the table this way. Let's review in the next meeting to validate. If the group wants to go back, we can do so. |
|
@jrayback — Please be careful about ambiguous date formats like |
|
@Hkdolts Support for the following hash functions has been added everywhere a hash function is used in a did:webplus-specific way, so that it's possible to choose whichever hash function is deemed appropriate. Previously supported:
Added:
|
Excellent - @vdods are you planning on presenting the updates to the WG soon? |
…nded Signed-off-by: Jonathan Rayback <jrayback@gmail.com>
|
@Hkdolts Here's a document showing use of each hash function across the board: https://github.com/LedgerDomain/did-webplus/blob/cee6751003c7f2f1597281f129adaaabacef5c62/doc/example-hash-function-selection.md I'll mention this in the WG call this week. |
|
@vdods , what is the status of |
|
@jrayback Uni resolver support is not yet finished. We're considering just making it part of the VDG service, since they basically play the same role. |
|
Does this method fulfill the "Multiple Impls" criterion? |
peacekeeper
left a comment
There was a problem hiding this comment.
I think there are several requirements in the "DIF-recommended" process which are not yet met by this method, and potentially also some misunderstandings/disagreements about the process itself that should be addressed first.
This concern was not raised within the 60-review period and, by policy, cannot deter |
|
Currently there is only the reference implementation, which implements the whole did:webplus spec. |
So, there's no known interop? Nor demonstration nor confirmation that the I see.... |
Mark did:webplus as DIF recommended.