Update did:solidus specification link to v0.2.0 - #741
Merged
Conversation
The registered v0.1.0 tag carries errors in the identifier grammar and its examples. v0.2.0 corrects them without changing the method.
fatih-koc
requested review from
KDean-Dolphin,
MizukiSonoko,
Steffytan,
adam-burns,
ajile-in,
apuchitnis,
gatemezing,
genaris,
msporny,
ottomorac,
rajivrajani,
stenreijers,
swcurran and
wip-abramson
as code owners
August 6, 2026 16:20
🤖 AI Preliminary Specification ReviewThis is an advisory, automated review of the DID Method specification(s) referenced in this pull request, checked against the registration checklist. It does not replace review by the registry editors.
|
| Item | Level | Status | Notes | |
|---|---|---|---|---|
| ✅ | M1. Specification is reachable and is a DID Method specification | MUST | pass | |
| ✅ | M2. DID Method Syntax is defined | MUST | pass | |
| ✅ | M3. CRUD operations are defined | MUST | pass | |
| ✅ | M4. Security Considerations section is present and substantive | MUST | pass | |
| ✅ | M5. Privacy Considerations section is present and substantive | MUST | pass | |
| ✅ | M6. Method name is indicative and non-generic | MUST | pass | |
| ✅ | M7. No unreasonable legal, security, moral, or privacy harms | MUST | pass | |
| ✅ | M8. Human-readable description of the addition | MUST | pass | |
| ✅ | S1. contactEmail present | OPTIONAL | pass | |
| ✅ | S2. verifiableDataRegistry present | OPTIONAL | pass | |
| S3. Intellectual-property posture is clear | SHOULD | warn | No explicit IP/copyright/license statement is visible in the fetched (truncated) text. This is advisory only; the Status section clarifies it is not a W3C endorsement but does not state licensing. |
swcurran
approved these changes
Aug 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the
specificationURL fordid:solidusfrom thev0.1.0tag tov0.2.0.v0.1.0— the version this registry currently points at — contains errors in the identifiergrammar and in every one of its examples. They were found while checking our own documentation
against the chain and are corrected in
v0.2.0:base58charwasALPHA / DIGIT, whichincludes
0,O,Iandl, with the exclusions noted only in a comment. A parser generatedfrom that grammar accepts identifiers
did:soliduscan never produce. The alphabet is nowenumerated as six explicit ranges.
decoded to 23 bytes rather than the specified 20; the other two were not Base58 at all (both
contained
0).a 20-byte payload encodes to 20–28 characters. That upper bound is what allowed the invalid
32-character examples to look acceptable. Over-28 is now a
MUSTreject, and the bound is in thegrammar (
20*28base58char).publicKeyMultibasedid not deriveto its own
id, so the single worked example of §4.2 contradicted it. It now uses a verified pair,and the identifier resolves on the public testnet.
No normative change to the method. The identifier construction, DID Document shape and operation
semantics are unchanged, and every identifier valid under
v0.1.0remains valid — this is a minorrevision under the spec's own versioning rule.
SPEC.mdcarries a "Changes in version 0.2.0" sectionfor implementers who generated a parser from
v0.1.0.Registered via #713. Contact details in the entry are unchanged; only the
specificationfield moves.