Beat [3/4]: Implement Consumer on chainWatcher and resolvers
#8922
Commits on Oct 15, 2024
-
-
contractcourt: add methods to decide spend path in
htlcSuccessResolverThis commit adds a few helper methods to decide how the htlc output should be spent.
-
contractcourt: add methods to send sweep requests in `htlcSuccessReso…
…lver` This commit is a pure refactor in which moves the sweep handling logic into the new methods.
-
contractcourt: add resolver handlers in
htlcSuccessResolverThis commit refactors the `Resolve` method by adding two resolver handlers to handle waiting for spending confirmations.
-
contractcourt: add
Launchmethod to anchor/breach resolverWe will use this and its following commits to break the original `Resolve` methods into two parts - the first part is moved to a new method `Launch`, which handles sending a sweep request to the sweeper. The second part remains in `Resolve`, which is mainly waiting for a spending tx. Breach resolver currently doesn't do anything in its `Launch` since the sweeping of justice outputs are not handled by the sweeper yet.
-
-
contractcourt: add
Launchmethod to htlc success resolverThis commit breaks the `Resolve` into two parts - the first part is moved into a `Launch` method that handles sending sweep requests, and the second part remains in `Resolve` which handles waiting for the spend.
-
contractcourt: add methods to handle sweep in
htlcTimeoutResolverThis commit adds new methods to handle making sweep requests based on the spending path used by the outgoing htlc output.
-
contractcourt: add methods to checkpoint states
This commit adds checkpoint methods in `htlcTimeoutResolver`, which are similar to those used in `htlcSuccessResolver`.
-
contractcourt: add methods to resolve different paths
This commit adds more methods to handle resolving the spending of the output based on different spending paths.
-
-
contractcourt: add
Launchmethod to incoming contest resolverA minor refactor is done to support implementing `Launch`.
-
-
-
-
chainio: enable spending txns lookup in
BlockbeatThis commit adds two interface methods, `HasOutpointSpent` and `HasOutpointSpentByScript` so the caller can use the current `blockbeat` to decide whether a given outpoint or a pkscript has been spent or not.
-
chainio+contractcourt: handle
blockbeatinchainWatcherThis commit adds a new method on `Blockbeat` to query the spending tx for a given outpoint and pkScript, thus treating `Blockbeat` as the single source of truth.
-
contractcourt: use
ShortChanIDinstead ofChannelPointin logThis way the log lines are shorten.
-
-
contractcourt: fix concurrent access to
resolvedThis commit makes `resolved` an atomic bool to avoid data race. This field is now defined in `contractResolverKit` to avoid code duplication.
-
-
multi: add new method
ChainArbitrator.RedispatchBlockbeatThis commit adds a new method to enable us resending the blockbeat in `ChainArbitrator`, which is needed for the channel restore as the chain watcher and channel arbitrator are added after the start of the chain arbitrator.
-
contractcourt: make sure output from legacy success tx is swept
This commit fixes an existing issue where the output from a pre-anchor type HTLC success tx is not swept.
-
-
-
This commit fixes the multi hop tests to reflect the latest sweeper changes.
-
itest: fix tests regarding the new sweeper behavior
This commit fix watchtower, channel backup, and route blinding tests re the new sweeper behavior.
