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

Register dnsValidationDomain property #566

Merged
merged 4 commits into from
Aug 26, 2024
Merged

Conversation

jessecarter111
Copy link
Contributor

@jessecarter111 jessecarter111 commented Jul 10, 2024

The "dnsDomain" property, as outlined in the High Assurance DIDs with DNS, helps to establishes a clear and verifiable link between a non web based DID and a DNS domain. This linkage leverages the existing trust infrastructure of the DNS, which is widely recognized and relied upon. By incorporating the "dnsDomain" property, non web based DIDs can benefit from the established trust model of DNS, providing an additional layer of assurance for entities interacting with the DID. It also enables the ability to associate a non web based DID with a DNS based identifier in accordance to the spec, without infringing upon the usage of the "alsoKnownAs" property or "linkedDomains" service type.


Preview | Diff

@peacekeeper
Copy link
Contributor

I think this is definitely useful, and the spec already looks very advanced..!

@apuchitnis apuchitnis self-requested a review July 10, 2024 18:01
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
@msporny msporny changed the title Added property to index Register dnsDomain property Jul 10, 2024
Copy link
Contributor

@davidlehn davidlehn left a comment

Choose a reason for hiding this comment

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

Sync names. PR title should be updated too.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
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.

LGTM once @davidlehn's changes are incorporated. You might want to rebase this branch before we merge it, @jessecarter111

@jessecarter111 jessecarter111 changed the title Register dnsDomain property Register dnsValidationDomain property Aug 15, 2024
Jesse and others added 4 commits August 15, 2024 16:20
@jessecarter111
Copy link
Contributor Author

Pinging for visibility on the workflow approval. Branch was rebased off main fork and requested changes added.

@msporny msporny merged commit 472c28a into w3c:main Aug 26, 2024
1 check passed
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.

5 participants