Skip to content

Commit 6f1f50f

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 b3a615f commit 6f1f50f

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
@@ -3225,6 +3225,7 @@ macro_rules! emit_initial_channel_ready_event {
32253225
channel_id: $channel.context.channel_id(),
32263226
user_channel_id: $channel.context.get_user_id(),
32273227
counterparty_node_id: $channel.context.get_counterparty_node_id(),
3228+
funding_txo: $channel.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
32283229
channel_type: $channel.funding.get_channel_type().clone(),
32293230
}, None));
32303231
$channel.context.set_initial_channel_ready_event_emitted();
@@ -9662,6 +9663,7 @@ This indicates a bug inside LDK. Please report this error at https://github.com/
96629663
channel_id: chan.context.channel_id(),
96639664
user_channel_id: chan.context.get_user_id(),
96649665
counterparty_node_id: chan.context.get_counterparty_node_id(),
9666+
funding_txo: chan.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
96659667
channel_type: chan.funding.get_channel_type().clone(),
96669668
}, None));
96679669

@@ -11821,6 +11823,7 @@ where
1182111823
channel_id: funded_channel.context.channel_id(),
1182211824
user_channel_id: funded_channel.context.get_user_id(),
1182311825
counterparty_node_id: funded_channel.context.get_counterparty_node_id(),
11826+
funding_txo: funded_channel.funding.get_funding_txo().map(|outpoint| outpoint.into_bitcoin_outpoint()),
1182411827
channel_type: funded_channel.funding.get_channel_type().clone(),
1182511828
}, None));
1182611829
}

0 commit comments

Comments
 (0)