-
Notifications
You must be signed in to change notification settings - Fork 56
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:dht to conform to the latest version of the specification #497
Comments
ok so in web5-js: ❌ - Not done
|
5 - looks right |
Number 3 draft pr - #502 |
… 2 compliance (#514) - Added support for publishing NS records and fixed bugs to achieve official test vector 2 compliance. - Removed kid in DNS records according to DID DHT spec update. - Minor renaming. - QoL - Updated CODEOWNERS to further increase review efficiency. - QoL - Added HTML code coverage output for `dids` repo for immediate coverage feedback.
1. Vector 3 compliance 2. X25519 support 3. Previous DID link support 4. DNS record chunking support for record > 255 characters (only in context of vector 3 compliance, will need to apply generically in a separate PR that addresses item 4 specifically in #497) 5. Some test refactoring
There have been a number of changes to the DID DHT spec since implementation, which I've captured below. These changes should be backwards compatible, meaning DIDs you use today will not break.
These changes are already in place in the DID DHT project itself, and you can find passing test vectors here as a reference.
Additionally, I would recommend:
republish
feature, which is able to take the latest state of a DID DHT document and re-publish it to a gateway.If not in practice already, it would make sense to expand our "PortableDID" construct to store histories of DID DHT documents and republish should only work on the latest version (seq num).
After this you can anticipate two additional changes to DID DHT pertaining to the following issues:
74 will be a breaking change, though (9) from above paves a path to make it less disruptive.
Please let me know if you have any questions.
The text was updated successfully, but these errors were encountered: