Create amt.json#639
Conversation
gatemezing
left a comment
There was a problem hiding this comment.
The proposal meets the minimum requirements
|
I joined the DID community, Lint check please. |
msporny
left a comment
There was a problem hiding this comment.
Please put more effort into your Security and Privacy Considerations section. It should at least have a few more items or at least a few paragraphs of the privacy and security implications of implementing your specification.
|
Agree that the minimum registration requirements have been met. Other comments from my review: It's not clear how a DID Resolver gets the information necessary to verify the DID -- presumably that is some sort of out of band process. The use of the term "Example" in Section 1.2.1 Create, step 3 should probably be removed. For the verification of the DID identifier to work, the JSON must match the template exactly so that when canonicalized and hashed, the correct hash is generated. It would be good to clean up things like that and (as @msporny notes) the Security and Privacy sections. The PR needs to be updated in order to merge it. |
There was a problem hiding this comment.
DID Method Registration
As a DID method registrant, I have ensured that my DID method registration complies with the following statements:
contactEmailaddress [OPTIONAL].verifiableDataRegistryentry [OPTIONAL].