Skip to content

Commit 03f632a

Browse files
jkczyzclaude
andcommitted
Emit SpliceFailed event when funded channels shut down with active splice negotiations
Adds SpliceFailed event emission immediately after ChannelClosed events when a FundedChannel is shut down while having an active splice negotiation. This ensures users are notified when splice operations are terminated due to channel closure. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 38425c8 commit 03f632a

File tree

2 files changed

+63
-1
lines changed

2 files changed

+63
-1
lines changed

lightning/src/ln/channel.rs

Lines changed: 50 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1188,6 +1188,9 @@ pub(crate) struct ShutdownResult {
11881188
pub(crate) unbroadcasted_funding_tx: Option<Transaction>,
11891189
pub(crate) channel_funding_txo: Option<OutPoint>,
11901190
pub(crate) last_local_balance_msat: u64,
1191+
/// If a splice was in progress when the channel was shut down, this contains
1192+
/// the splice funding information for emitting a SpliceFailed event.
1193+
pub(crate) splice_funding_failed: Option<SpliceFundingFailed>,
11911194
}
11921195

11931196
/// Tracks the transaction number, along with current and next commitment points.
@@ -2676,6 +2679,15 @@ pub(crate) struct SpliceInstructions {
26762679
locktime: u32,
26772680
}
26782681

2682+
impl SpliceInstructions {
2683+
fn into_contributed_inputs_and_outputs(self) -> (Vec<bitcoin::OutPoint>, Vec<TxOut>) {
2684+
(
2685+
self.our_funding_inputs.into_iter().map(|input| input.utxo.outpoint).collect(),
2686+
self.our_funding_outputs,
2687+
)
2688+
}
2689+
}
2690+
26792691
impl_writeable_tlv_based!(SpliceInstructions, {
26802692
(1, adjusted_funding_contribution, required),
26812693
(3, our_funding_inputs, required_vec),
@@ -6030,6 +6042,7 @@ where
60306042
is_manual_broadcast: self.is_manual_broadcast,
60316043
channel_funding_txo: funding.get_funding_txo(),
60326044
last_local_balance_msat: funding.value_to_self_msat,
6045+
splice_funding_failed: None,
60336046
}
60346047
}
60356048

@@ -6823,7 +6836,42 @@ where
68236836
}
68246837

68256838
pub fn force_shutdown(&mut self, closure_reason: ClosureReason) -> ShutdownResult {
6826-
self.context.force_shutdown(&self.funding, closure_reason)
6839+
let splice_funding_failed = self
6840+
.pending_splice
6841+
.as_mut()
6842+
.and_then(|pending_splice| pending_splice.funding_negotiation.take())
6843+
.filter(|funding_negotiation| funding_negotiation.is_initiator())
6844+
.map(|_funding_negotiation| {
6845+
// FIXME: Populte after #4120 is merged
6846+
SpliceFundingFailed {
6847+
funding_txo: todo!(),
6848+
channel_type: todo!(),
6849+
contributed_inputs: todo!(),
6850+
contributed_outputs: todo!(),
6851+
}
6852+
})
6853+
.or_else(|| {
6854+
self.quiescent_action.take().and_then(|quiescent_action| match quiescent_action {
6855+
QuiescentAction::Splice(instructions) => {
6856+
let (inputs, outputs) = instructions.into_contributed_inputs_and_outputs();
6857+
Some(SpliceFundingFailed {
6858+
funding_txo: None,
6859+
channel_type: None,
6860+
contributed_inputs: inputs,
6861+
contributed_outputs: outputs,
6862+
})
6863+
},
6864+
#[cfg(any(test, fuzzing))]
6865+
_ => {
6866+
self.quiescent_action = Some(quiescent_action);
6867+
None
6868+
},
6869+
})
6870+
});
6871+
6872+
let mut shutdown_result = self.context.force_shutdown(&self.funding, closure_reason);
6873+
shutdown_result.splice_funding_failed = splice_funding_failed;
6874+
shutdown_result
68276875
}
68286876

68296877
fn interactive_tx_constructor_mut(&mut self) -> Option<&mut InteractiveTxConstructor> {
@@ -10258,6 +10306,7 @@ where
1025810306
is_manual_broadcast: self.context.is_manual_broadcast,
1025910307
channel_funding_txo: self.funding.get_funding_txo(),
1026010308
last_local_balance_msat: self.funding.value_to_self_msat,
10309+
splice_funding_failed: None,
1026110310
}
1026210311
}
1026310312

lightning/src/ln/channelmanager.rs

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4509,6 +4509,19 @@ where
45094509
last_local_balance_msat: Some(shutdown_res.last_local_balance_msat),
45104510
}, None));
45114511

4512+
// Emit SpliceFailed event immediately after ChannelClosed if there was an active splice negotiation
4513+
if let Some(splice_funding_failed) = shutdown_res.splice_funding_failed.take() {
4514+
pending_events.push_back((events::Event::SpliceFailed {
4515+
channel_id: shutdown_res.channel_id,
4516+
counterparty_node_id: shutdown_res.counterparty_node_id,
4517+
user_channel_id: shutdown_res.user_channel_id,
4518+
funding_txo: splice_funding_failed.funding_txo,
4519+
channel_type: splice_funding_failed.channel_type,
4520+
contributed_inputs: splice_funding_failed.contributed_inputs,
4521+
contributed_outputs: splice_funding_failed.contributed_outputs,
4522+
}, None));
4523+
}
4524+
45124525
if let Some(transaction) = shutdown_res.unbroadcasted_funding_tx {
45134526
let funding_info = if shutdown_res.is_manual_broadcast {
45144527
FundingInfo::OutPoint {

0 commit comments

Comments
 (0)