-
Notifications
You must be signed in to change notification settings - Fork 97
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
Should DID syntax allow an empty "method-specific-id"? #34
Comments
I've been thinking about this for a while and I now have a slight preference, but I'd like to share some considerations first. What is the DID subject in the case of an empty method-specific identifier? The answer that was given in the past was that the DID identifies the DID method or DID registry itself. But then, what does it mean...
I admit that some of the above sounds intriguing and could make sense. But on the other hand, an empty method-specific identifier feels a bit like a special case that is quite different from what DIDs have been designed for. What I'd like to avoid is allowing this merely for being able to discover metadata about a DID resolver, or for some kind of general-purpose query mechanism to the DID registry. Those things should be handled differently, not by piggybacking on DID syntax and DID resolution. Instead, the question we need to answer here is "does it make sense for the DID registry to be considered a DID subject itself". Thoughts? |
Since there has been no activity on this issue, my (slight) preference would be to NOT allow an empty So unless anyone feels differently, I propose to raise a PR to change the ABNF to NOT allow an empty |
No, the only use case for it is weak and can be solved through a variety of other mechanisms like a regex or matching against "did:v1" or a variety of other things that are perfectly fine and don't need to comply w/ DID syntax. |
Thanks @msporny I agree. Others please add thoughts. If there is no opinion to the contrary, we can create a PR to change the ABNF to no longer allow an empty |
Agree with @msporny. |
PR looks good and I agree with the consensus that's been reached. |
This has been addressed by #216. Closing. |
@peacekeeper moved from CCG (w3c-ccg/did-spec#198)
The text was updated successfully, but these errors were encountered: