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

CHIP-0004: DID1 Standard #18

Merged
merged 18 commits into from
Feb 1, 2023
Merged

CHIP-0004: DID1 Standard #18

merged 18 commits into from
Feb 1, 2023

Conversation

danieljperry
Copy link
Contributor

No description provided.

Copy link

@ddesiderio ddesiderio left a comment

Choose a reason for hiding this comment

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

Thanks for getting this CHIP started to define the standard for DID1. I've added a few preliminary discussion points related to the spec for your review. Related to my previous CHIP request I have also asked a few questions around metadata and respective schema which would be great to solve before DID1 release.

Looking forward to seeing some more detail on the RPCs once this gets fleshed out a bit further 😄

* For recovery, each DID should contain a hash of a list of trusted DIDs. If recovery is not a desired feature, then a placeholder may be used instead
* The trusted DID list may contain an unbounded number of entries
* DIDs must support transaction fees upon transferal
* DIDs should not contain any personally identifiable data

Choose a reason for hiding this comment

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

For DIDs to have real-world usability they will need to have some way to link to PII info at some level. I agree with the suggestion of avoiding writing any PII data directly on chain (if at all possible), but DIDs will need some other way (external metadata links, etc.) to have ties to PII info. Consider also the KYC case which will likely require 3rd parties to "stamp" a DID with some sort of verified info, keeping KYC info off-chain somewhere linked to the DID.

CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
@wriches wriches changed the title Open DID1 CHIP CHIP-0004: DID1 Standard Jun 15, 2022
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
@wriches wriches assigned wriches and danieljperry and unassigned wriches Jun 26, 2022
@CommanderMoto
Copy link

CommanderMoto commented Jun 30, 2022

I noticed that the Profile I got from my spanking new Chia 1.4.0 client uses the format "did:chia:xxxxxxx" - is it a good idea for testnet dID to look the same as mainnet dID?

Had this thought while reviewing DIF sidetree specification: https://identity.foundation/sidetree/spec/#did-uri-composition

Signed-off-by: danieljperry <d.perry@chia.net>
Signed-off-by: danieljperry <d.perry@chia.net>
Signed-off-by: danieljperry <d.perry@chia.net>
hoffmang9
hoffmang9 previously approved these changes Dec 15, 2022
Copy link
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: danieljperry <d.perry@chia.net>
CHIPs/chip-0004.md Outdated Show resolved Hide resolved
@danieljperry
Copy link
Contributor Author

This CHIP is now Final. Further changes (other than errata) are no longer allowed.

Copy link
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

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

lgtm

@danieljperry danieljperry merged commit 4f9ce25 into main Feb 1, 2023
@danieljperry danieljperry deleted the did1 branch February 1, 2023 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

8 participants