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

Standardize RData delimiter #83

Closed
mistermoe opened this issue Jan 3, 2024 · 5 comments
Closed

Standardize RData delimiter #83

mistermoe opened this issue Jan 3, 2024 · 5 comments
Assignees

Comments

@mistermoe
Copy link
Member

image

Is there a specific reason as to why the root record's RData delimits fields using ; but _kN and _sN uses ,? i think we should use ; to delimit fields so that we can use , for multiple values for a given field.

If not, there are typos here:

image
  • _s0._did.'s RData value should be t=LinkedDomains,uri=foo.com,… not t=LinkedDomains;uri=foo.com;…
  • _s1._did has the same issue
@decentralgabe
Copy link
Member

; separates distinct entities (like different verification relationships) whereas , delineates multiple values for a given field

there is indeed a typo below, which I will address.

@mistermoe
Copy link
Member Author

yeah in that case i think _sN._did. should absolutely use ; to separate entities vs. , as you have right now because values like uri can have multiple values as noted here.

image

On that note, did did:dht make an explicit decision to not support serviceEndpoint being an object? DWNs require this which you can see here

image

@decentralgabe
Copy link
Member

@mistermoe the spec here creates a profile (method) of DID core. so even though you're correct that DID core supports multiple values, the spec as written does not.

I think we should close this issue (after #84 is merged) and open a new issue to discuss services with multiple values.

@decentralgabe
Copy link
Member

fixed in #84

@mistermoe
Copy link
Member Author

sounds good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants