decentralized-identity / peer-did-method-spec

A rich DID method that has no blockchain dependencies. The verifiable data registry is a synchronization protocol between peers.
https://decentralized-identity.github.io/peer-did-method-spec/index.html
Apache License 2.0
27 stars 17 forks source link

Consider removing dynamic layer #45

Open brianorwhatever opened 1 year ago

brianorwhatever commented 1 year ago

As far as I can tell, none of the implementations of this spec use the dynamic layer of this method. If everyone is just using static this method could become quite succinct which would help adoption.

dhh1128 commented 1 year ago

I agree. However, I don't want to do all the surgery to pull out the dynamic stuff. Are you interested in writing a PR and adding yourself to the contributors list?

brianorwhatever commented 1 year ago

I would if we are going to revive this method. If not I don't think it's worth the effort. At this stage I'm just gathering information to figure out the best plan moving forward.

brianorwhatever commented 1 year ago

Ultimately, this method to me is entirely captured by static and numalgo=2. Here is the reason I believe this to be true..

static:

dynamic:

So that being said I am proposing removing 5/6 of the methods available in this spec which is quite a drastic change.. I could see creating a separate "did:peer dynamic extension" spec being of value if there is interest in that but as experience is showing there hasn't really been so far..

dhh1128 commented 1 year ago

I am not opposed to most of those suggestions, @brianorwhatever . I think there are people using numalgo 0, so removing might disrupt them -- but everything else would probably be easy to remove to simplify.

brianorwhatever commented 1 year ago

yeah, that is what I suspect as well which is unfortunate as it replicates did:key but should probably remain in as you say. Do you know of anyone using numalgo 1?

dhh1128 commented 1 year ago

re numalgo 1: no, I believe it's unimplemented

brianorwhatever commented 1 year ago

I have recently discovered https://impervious.ai/ which in its backup recovery key and throughout the app uses numalgo with an initialState param