search

decentralized-identity / sidetree

Sidetree Specification and Reference Implementation
https://identity.foundation/sidetree/spec
Apache License 2.0
438 stars 112 forks source link

Remove method prefix from initial state #770

Closed OR13 closed 4 years ago

OR13 commented 4 years ago

addresses https://github.com/decentralized-identity/sidetree/issues/769

Removes the method prefix from initial-state did parameter.

jmandel commented 4 years ago

I missed a window to comment since it looks this was created and merged within an hour, but this is a breaking change to long-form DIDs. I thought our intention was to avoid making breaking changes to long-form DIDs at this stage. The PR here and issue at https://github.com/decentralized-identity/sidetree/issues/769 don't explain the rationale.

If I'm missing something, please let me know. (I'd prefer to see no change, or a spec that ensures -ion-initial-state remains acceptable even if initial-state is preferred.)

csuwildcat commented 4 years ago

@jmandel we can make this a backwards compatible change - I will ensure it still works with the prefix in ION.

jmandel commented 4 years ago

Thanks! Is there a spot where we document these kinds of additional rules for ion?

csuwildcat commented 4 years ago

@jmandel I will get together an ION spec that is mostly refs to Sidetree, but with the bits that are specific to it. That's one of my deliverables for July