nuts-foundation / nuts-specification

Contains the source of the Nuts specification RFCs.
https://nuts-foundation.gitbook.io
2 stars 0 forks source link

changed requirement of TX to point to all prevs #198

Closed woutslakhorst closed 2 years ago

woutslakhorst commented 2 years ago

See appendix A.2 of rfc004 for rationale

reinkrul commented 2 years ago

Just thought of additional issues:

I see potential issues when nobody tries to merge unmerged heads:

woutslakhorst commented 2 years ago

There's no incentive to limit the number of branches. While a single branch, or a bunch of them are harmless, 1000 branches with a TX with the same LC is not (due to page size). Now the number of branches can be every-increasing, while previously it would attempt to return to 1 branch

When branches are merged, there's also no limit on the number of branches.

A VC TX does not refer to the TX which created/last updated the subject's DID document, so it might be in a different branch

Yes it does: network_publisher#88 All dependencies on earlier transactions must be solved by prevs, ordering of transactions can never be used because that would be an attack vector.