sidetree

sidetree copied to clipboard

Possible designs: 1. Separate APIs for validating operations/patches. 1. Part of standard response as resolver metadata.

thehenrytsai

Publish CAS files only after anchor string is anchored on ledger. This would eliminate the possibility of a middleman observing the ledger mempool for ION transactions being able to obtain...

thehenrytsai

isaacJChen

Consider removing the commit-reveal checks in `OperationProcessor`.

2

Technically, the commit-reveal checks is already performed by `Resolver` because it builds the commit-reveal lookup map, so the commit-reveal checks in `OperationProcessor` is current redundant. One could make a case...

thehenrytsai

Today the bitcoin service is required to perform full transaction processing until the current block height during initialization, and if an unexpected error is thrown (such as dependent service being...

thehenrytsai

Currently the bitcoin service logs an non-critical event if value time lock is not configured as if something bad happened: { Error: {"code":"value_time_lock_not_found"} at BitcoinProcessor.getActiveValueTimeLockForThisNode (/datadrive/ion/node_modules/@decentralized-identity/sidetree/dist/lib/bitcoin/BitcoinProcessor.js:250:19) at Object. (/datadrive/ion/dist/src/bitcoin.js:105:106) at...

thehenrytsai

A published package right now includes all files, including all the docs, ts source, json, tests, etc. We should: 1. Reduce the published package by including only compiled file (*.js)...

thehenrytsai

See: - https://github.com/w3c-ccg/did-method-registry/issues/54 - https://github.com/w3c-ccg/did-method-registry/pull/84 - https://github.com/decentralized-identity/element/blob/master/docs/did-method-spec/spec.md

OR13

https://github.com/decentralized-identity/sidetree/blob/a8ba6ec2a1fb809df55ce54e47339d8ac1e7df34/lib/bitcoin/BitcoinClient.ts#L47 If the admin forgets to add in either the import string or an actual IBitcoinWallet on this parameter then we should throw here. Currently, the `this.bitcoinWallet` member stays undefined...

mudiali

And deprecate API definitions in protocol.md

thehenrytsai