Skip to content

Commit c6e3515

Browse files
committed
Add funding_txo to ChannelReady event
A ChannelReady event is used for both channel establishment and splicing to indicate that the funding transaction is confirmed to an acceptable depth and thus the channel can be used with the funding. An upcoming SplicePending event will be emitted for each pending splice (i.e., both the initial splice attempt and any RBF attempts). Thus, when a ChannelReady event is emitted, the funding_txo must be included to differentiate between which ChannelPending -- which also contains the funding_txo -- that the event corresponds to.
1 parent 60e21ac commit c6e3515

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

lightning/src/events/mod.rs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1378,6 +1378,11 @@ pub enum Event {
13781378
user_channel_id: u128,
13791379
/// The `node_id` of the channel counterparty.
13801380
counterparty_node_id: PublicKey,
1381+
/// The outpoint of the channel's funding transaction.
1382+
///
1383+
/// Will be `None` if the channel's funding transaction reached an acceptable depth prior to
1384+
/// version 0.2.
1385+
funding_txo: Option<OutPoint>,
13811386
/// The features that this channel will operate with.
13821387
channel_type: ChannelTypeFeatures,
13831388
},
@@ -1929,11 +1934,13 @@ impl Writeable for Event {
19291934
ref channel_id,
19301935
ref user_channel_id,
19311936
ref counterparty_node_id,
1937+
ref funding_txo,
19321938
ref channel_type,
19331939
} => {
19341940
29u8.write(writer)?;
19351941
write_tlv_fields!(writer, {
19361942
(0, channel_id, required),
1943+
(1, funding_txo, option),
19371944
(2, user_channel_id, required),
19381945
(4, counterparty_node_id, required),
19391946
(6, channel_type, required),
@@ -2440,9 +2447,11 @@ impl MaybeReadable for Event {
24402447
let mut channel_id = ChannelId::new_zero();
24412448
let mut user_channel_id: u128 = 0;
24422449
let mut counterparty_node_id = RequiredWrapper(None);
2450+
let mut funding_txo = None;
24432451
let mut channel_type = RequiredWrapper(None);
24442452
read_tlv_fields!(reader, {
24452453
(0, channel_id, required),
2454+
(1, funding_txo, option),
24462455
(2, user_channel_id, required),
24472456
(4, counterparty_node_id, required),
24482457
(6, channel_type, required),
@@ -2452,6 +2461,7 @@ impl MaybeReadable for Event {
24522461
channel_id,
24532462
user_channel_id,
24542463
counterparty_node_id: counterparty_node_id.0.unwrap(),
2464+
funding_txo,
24552465
channel_type: channel_type.0.unwrap(),
24562466
}))
24572467
};

lightning/src/ln/channelmanager.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3228,6 +3228,7 @@ macro_rules! emit_initial_channel_ready_event {
32283228
channel_id: $channel.context.channel_id(),
32293229
user_channel_id: $channel.context.get_user_id(),
32303230
counterparty_node_id: $channel.context.get_counterparty_node_id(),
3231+
funding_txo: $channel.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
32313232
channel_type: $channel.funding.get_channel_type().clone(),
32323233
}, None));
32333234
$channel.context.set_initial_channel_ready_event_emitted();
@@ -9665,6 +9666,7 @@ This indicates a bug inside LDK. Please report this error at https://github.com/
96659666
channel_id: chan.context.channel_id(),
96669667
user_channel_id: chan.context.get_user_id(),
96679668
counterparty_node_id: chan.context.get_counterparty_node_id(),
9669+
funding_txo: chan.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
96689670
channel_type: chan.funding.get_channel_type().clone(),
96699671
}, None));
96709672

@@ -11835,6 +11837,7 @@ where
1183511837
channel_id: funded_channel.context.channel_id(),
1183611838
user_channel_id: funded_channel.context.get_user_id(),
1183711839
counterparty_node_id: funded_channel.context.get_counterparty_node_id(),
11840+
funding_txo: funded_channel.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
1183811841
channel_type: funded_channel.funding.get_channel_type().clone(),
1183911842
}, None));
1184011843
}

0 commit comments

Comments
 (0)