Skip to content

Commit ab9dfbc

Browse files
committed
Dual-advertise splice prototype and production feature bits
LDK 0.2.2 will rename the splice feature from prototype bit 155 to production bit 63. Setting both bits during the migration window keeps splice working with three peer cohorts on the same LSP binary: - un-upgraded clients on bit 155 only, - upgraded clients and ACINQ on bit 63, - upgraded clients that still see bit 155 (no harm). ACINQ is the exception: their eclair stack chokes when both bits are set, so the previous commit strips bit 155 from outbound Inits to ACINQ specifically. Sunset path: when all clients have upgraded, drop the set_splicing_optional() call and re-bump. Refs MDK-799
1 parent 56321de commit ab9dfbc

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

lightning/src/ln/channelmanager.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15746,6 +15746,9 @@ pub fn provided_init_features(config: &UserConfig) -> InitFeatures {
1574615746
features.set_simple_close_optional();
1574715747
features.set_quiescence_optional();
1574815748
features.set_splicing_optional();
15749+
// Dual-advertise alongside the prototype bit during the LDK 0.2.2 migration.
15750+
// Stripped per-peer for ACINQ in the BaseMessageHandler impl above. See MDK-799.
15751+
features.set_splicing_production_optional();
1574915752

1575015753
if config.channel_handshake_config.negotiate_anchors_zero_fee_htlc_tx {
1575115754
features.set_anchors_zero_fee_htlc_tx_optional();

0 commit comments

Comments
 (0)