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

Clarify that the DID Method of a witness DID and diffentiating the DID Log Entry proofs #123

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

swcurran
Copy link
Collaborator

Addresses #114 and clarifies that a witness DID SHOULD be a did:tdw and why. Also mentions the possiblity of mutual witnesses causing an infinite loop.

Signed-off-by: Stephen Curran swcurran@gmail.com

…D Log Entry proofs

Signed-off-by: Stephen Curran <swcurran@gmail.com>
@swcurran swcurran requested review from a team October 24, 2024 23:48
@@ -895,6 +903,12 @@ An overview of the [[ref: witness]] mechanism is as follows:
- Over time, the list of [[ref: witnesses]] may evolve, with each change being
approved by the declared list of [[ref: witnesses]] from **before** such a
change.
- The DID of the witnesses **SHOULD** be `did:tdw` DIDs to prevent the need for
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be a MUST?

@swcurran
Copy link
Collaborator Author

swcurran commented Nov 7, 2024

@brianorwhatever -- updated the DID, changing both SHOULDs to MUSTs and cleaned up the infinite loop wording. I think the second MUST is also needed as a resolver that gets into an infinite loop isn't very useful.

@swcurran swcurran marked this pull request as draft November 20, 2024 16:59
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.

2 participants