BlockchainWriter Confirmations

[lautarodragan]

Umbrella issue to track all changes needed to fully support blockchain confirmations.

Blockchain Confirmations RDD

Forks

• Warning on Blockchain Reorganization, implemented in https://github.com/poetapp/node/pull/672
• Warning on Blockchain Reorganization — Integration Test, implemented in https://github.com/poetapp/node/pull/867
• Reorganize (TBD)
  • Deliverable 1: Unconfirm
  • BlockchainReader: publish chainReorganized(blockHashNowStale, blockHeightNowStale)
  • View: onReorg => update .timestamp
  • Deliverable 2: Reconfirm
  • BlockchainReader Walk back over chain until split
  • BlockchainWriter: Rebroadcast transactions as needed
  • View: update .timestamp

Transaction Retries

• Transaction Retries (Naive)
  • Blockchain Writer - Store the date of transaction creation, implemented in https://github.com/poetapp/node/pull/735
• Transaction Retries (RBF) (TBD)

Note

Need to test whether orphaned transactions can be rebroadcasted with no modification.

See Timelock Defense Against Fee Sniping and Discourage fee sniping with nLockTime.

[geoffturk]

needs #157