mirrors/rust-lightning
1
0
Fork 0
mirror of https://github.com/lightningdevkit/rust-lightning.git synced 2025-02-24 23:08:36 +01:00
Code Issues Projects Releases Packages Wiki Activity

Improve Confirm docs

Browse source
This commit is contained in:
Elias Rohrer 2022-12-05 17:55:47 +01:00
parent de2acc0ee0
commit 9de45ce2d9
No known key found for this signature in database
GPG key ID: 36153082BDF676FD
1 changed files with 36 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

68
lightning/src/chain/mod.rs
View file

@ -107,31 +107,35 @@ pub trait Listen {
fn block_disconnected(&self, header: &BlockHeader, height: u32); fn block_disconnected(&self, header: &BlockHeader, height: u32);
} }
/// The `Confirm` trait is used to notify when transactions have been confirmed on chain or /// The `Confirm` trait is used to notify LDK when relevant transactions have been confirmed on
/// unconfirmed during a chain reorganization. /// chain or unconfirmed during a chain reorganization.
/// ///
/// Clients sourcing chain data using a transaction-oriented API should prefer this interface over /// Clients sourcing chain data using a transaction-oriented API should prefer this interface over
/// [`Listen`]. For instance, an Electrum client may implement [`Filter`] by subscribing to activity /// [`Listen`]. For instance, an Electrum-based transaction sync implementation may implement
/// related to registered transactions and outputs. Upon notification, it would pass along the /// [`Filter`] to subscribe to relevant transactions and unspent outputs it should monitor for
/// matching transactions using this interface. /// on-chain activity. Then, it needs to notify LDK via this interface upon observing any changes
/// with reference to the confirmation status of the monitored objects.
/// ///
/// # Use /// # Use
///
/// The intended use is as follows: /// The intended use is as follows:
/// - Call [`transactions_confirmed`] to process any on-chain activity of interest. /// - Call [`transactions_confirmed`] to notify LDK whenever any of the registered transactions or
/// - Call [`transaction_unconfirmed`] to process any transaction returned by [`get_relevant_txids`] /// outputs are, respectively, confirmed or spent on chain.
/// that has been reorganized out of the chain. /// - Call [`transaction_unconfirmed`] to notify LDK whenever any transaction returned by
/// - Call [`best_block_updated`] whenever a new chain tip becomes available. /// [`get_relevant_txids`] is no longer confirmed in the block with the given block hash.
/// - Call [`best_block_updated`] to notify LDK whenever a new chain tip becomes available.
/// ///
/// # Order /// # Order
/// ///
/// Clients must call these methods in chain order. Specifically: /// Clients must call these methods in chain order. Specifically:
/// - Transactions confirmed in a block must be given before transactions confirmed in a later /// - Transactions which are confirmed in a particular block must be given before transactions
/// block. /// confirmed in a later block.
/// - Dependent transactions within the same block must be given in topological order, possibly in /// - Dependent transactions within the same block must be given in topological order, possibly in
/// separate calls. /// separate calls.
/// - Unconfirmed transactions must be given after the original confirmations and before any /// - All unconfirmed transactions must be given after the original confirmations and before *any*
/// reconfirmation. /// reconfirmations, i.e., [`transactions_confirmed`] and [`transaction_unconfirmed`] calls should
/// never be interleaved, but always conduced *en bloc*.
/// - Any reconfirmed transactions need to be explicitly unconfirmed before they are reconfirmed
/// in regard to the new block.
/// ///
/// See individual method documentation for further details. /// See individual method documentation for further details.
/// ///
@ -140,9 +144,9 @@ pub trait Listen {
/// [`best_block_updated`]: Self::best_block_updated /// [`best_block_updated`]: Self::best_block_updated
/// [`get_relevant_txids`]: Self::get_relevant_txids /// [`get_relevant_txids`]: Self::get_relevant_txids
pub trait Confirm { pub trait Confirm {
/// Processes transactions confirmed in a block with a given header and height. /// Notifies LDK of transactions confirmed in a block with a given header and height.
/// ///
/// Should be called for any transactions registered by [`Filter::register_tx`] or any /// Must be called for any transactions registered by [`Filter::register_tx`] or any
/// transactions spending an output registered by [`Filter::register_output`]. Such transactions /// transactions spending an output registered by [`Filter::register_output`]. Such transactions
/// appearing in the same block do not need to be included in the same call; instead, multiple /// appearing in the same block do not need to be included in the same call; instead, multiple
/// calls with additional transactions may be made so long as they are made in [chain order]. /// calls with additional transactions may be made so long as they are made in [chain order].
@ -154,36 +158,36 @@ pub trait Confirm {
/// [chain order]: Confirm#order /// [chain order]: Confirm#order
/// [`best_block_updated`]: Self::best_block_updated /// [`best_block_updated`]: Self::best_block_updated
fn transactions_confirmed(&self, header: &BlockHeader, txdata: &TransactionData, height: u32); fn transactions_confirmed(&self, header: &BlockHeader, txdata: &TransactionData, height: u32);
/// Notifies LDK of a transaction that is no longer confirmed as result of a chain reorganization.
/// Processes a transaction that is no longer confirmed as result of a chain reorganization.
/// ///
/// Should be called for any transaction returned by [`get_relevant_txids`] if it has been /// Must be called for any transaction returned by [`get_relevant_txids`] if it has been
/// reorganized out of the best chain. Once called, the given transaction will not be returned /// reorganized out of the best chain or if it is no longer confirmed in the block with the
/// given block hash. Once called, the given transaction will not be returned
/// by [`get_relevant_txids`], unless it has been reconfirmed via [`transactions_confirmed`]. /// by [`get_relevant_txids`], unless it has been reconfirmed via [`transactions_confirmed`].
/// ///
/// [`get_relevant_txids`]: Self::get_relevant_txids /// [`get_relevant_txids`]: Self::get_relevant_txids
/// [`transactions_confirmed`]: Self::transactions_confirmed /// [`transactions_confirmed`]: Self::transactions_confirmed
fn transaction_unconfirmed(&self, txid: &Txid); fn transaction_unconfirmed(&self, txid: &Txid);
/// Notifies LDK of an update to the best header connected at the given height.
/// Processes an update to the best header connected at the given height.
/// ///
/// Should be called when a new header is available but may be skipped for intermediary blocks /// Must be called whenever a new chain tip becomes available. May be skipped for intermediary
/// if they become available at the same time. /// blocks.
fn best_block_updated(&self, header: &BlockHeader, height: u32); fn best_block_updated(&self, header: &BlockHeader, height: u32);
/// Returns transactions that must be monitored for reorganization out of the chain along
/// Returns transactions that should be monitored for reorganization out of the chain along /// with the hash of the block as part of which it had been previously confirmed.
/// with the hash of the block as part of which had been previously confirmed.
/// ///
/// Will include any transactions passed to [`transactions_confirmed`] that have insufficient /// Will include any transactions passed to [`transactions_confirmed`] that have insufficient
/// confirmations to be safe from a chain reorganization. Will not include any transactions /// confirmations to be safe from a chain reorganization. Will not include any transactions
/// passed to [`transaction_unconfirmed`], unless later reconfirmed. /// passed to [`transaction_unconfirmed`], unless later reconfirmed.
/// ///
/// May be called to determine the subset of transactions that must still be monitored for /// Must be called to determine the subset of transactions that must be monitored for
/// reorganization. Will be idempotent between calls but may change as a result of calls to the /// reorganization. Will be idempotent between calls but may change as a result of calls to the
/// other interface methods. Thus, this is useful to determine which transactions may need to be /// other interface methods. Thus, this is useful to determine which transactions must be
/// given to [`transaction_unconfirmed`]. If any of the returned transactions are confirmed in /// given to [`transaction_unconfirmed`].
/// a block other than the one with the given hash, they need to be unconfirmed and reconfirmed ///
/// via [`transaction_unconfirmed`] and [`transactions_confirmed`], respectively. /// If any of the returned transactions are confirmed in a block other than the one with the
/// given hash, they need to be unconfirmed and reconfirmed via [`transaction_unconfirmed`] and
/// [`transactions_confirmed`], respectively.
/// ///
/// [`transactions_confirmed`]: Self::transactions_confirmed /// [`transactions_confirmed`]: Self::transactions_confirmed
/// [`transaction_unconfirmed`]: Self::transaction_unconfirmed /// [`transaction_unconfirmed`]: Self::transaction_unconfirmed