Log confirmation of channel funding transaction

Author: jkczyz

lightning/src/ln/channel.rs

@@ -5063,10 +5063,13 @@ impl<SP: Deref> ChannelContext<SP> where SP::Target: SignerProvider {
 		self.get_initial_counterparty_commitment_signature(funding, logger)
 	}
 
-	fn check_for_funding_tx_confirmed(
+	fn check_for_funding_tx_confirmed<L: Deref>(
 		&mut self, funding: &mut FundingScope, block_hash: &BlockHash, height: u32,
-		index_in_block: usize, tx: &mut ConfirmedTransaction,
-	) -> Result<bool, ClosureReason> {
+		index_in_block: usize, tx: &mut ConfirmedTransaction, logger: &L,
+	) -> Result<bool, ClosureReason>
+	where
+		L::Target: Logger,
+	{
 		let funding_txo = match funding.get_funding_txo() {
 			Some(funding_txo) => funding_txo,
 			None => {
@@ -5116,6 +5119,14 @@ impl<SP: Deref> ChannelContext<SP> where SP::Target: SignerProvider {
 						Err(_) => panic!("Block was bogus - either height was > 16 million, had > 16 million transactions, or had > 65k outputs"),
 					};
 
+					log_info!(
+						logger,
+						"Funding txid {} for channel {} confirmed in block {}",
+						funding_txo.txid,
+						&self.channel_id(),
+						block_hash,
+					);
+
 					return Ok(true);
 				}
 			}
@@ -8686,7 +8697,7 @@ impl<SP: Deref> FundedChannel<SP> where
 			// and send it immediately instead of waiting for a best_block_updated call (which may have
 			// already happened for this block).
 			let is_funding_tx_confirmed = self.context.check_for_funding_tx_confirmed(
-				&mut self.funding, block_hash, height, index_in_block, &mut confirmed_tx,
+				&mut self.funding, block_hash, height, index_in_block, &mut confirmed_tx, logger,
 			)?;
 
 			if is_funding_tx_confirmed {
@@ -8718,7 +8729,7 @@ impl<SP: Deref> FundedChannel<SP> where
 			#[cfg(splicing)]
 			for (index, funding) in self.pending_funding.iter_mut().enumerate() {
 				if self.context.check_for_funding_tx_confirmed(
-					funding, block_hash, height, index_in_block, &mut confirmed_tx,
+					funding, block_hash, height, index_in_block, &mut confirmed_tx, logger,
 				)? {
 					if funding_already_confirmed || confirmed_funding_index.is_some() {
 						let err_reason = "splice tx of another pending funding already confirmed";