Rename InboundOnionPayload::Receive::outgoing_cltv_value
authorValentine Wallace <vwallace@protonmail.com>
Tue, 12 Dec 2023 22:45:46 +0000 (17:45 -0500)
committerValentine Wallace <vwallace@protonmail.com>
Wed, 10 Jan 2024 15:56:24 +0000 (10:56 -0500)
.. since there is no outgoing cltv for received HTLCs.

lightning/src/ln/channelmanager.rs
lightning/src/ln/msgs.rs
lightning/src/ln/onion_payment.rs

index 8ba0b52e8a966b7797cfcc4077389e185af86d75..5fe0ad3d69445b00e6077425898628f138fd4dfd 100644 (file)
@@ -12128,7 +12128,7 @@ mod tests {
                let extra_fee_msat = 10;
                let hop_data = msgs::InboundOnionPayload::Receive {
                        amt_msat: 100,
-                       outgoing_cltv_value: 42,
+                       cltv_expiry_height: 42,
                        payment_metadata: None,
                        keysend_preimage: None,
                        payment_data: Some(msgs::FinalOnionHopData {
@@ -12150,7 +12150,7 @@ mod tests {
                // If amt_received + extra_fee is equal to the sender intended amount, we're fine.
                let hop_data = msgs::InboundOnionPayload::Receive { // This is the same payload as above, InboundOnionPayload doesn't implement Clone
                        amt_msat: 100,
-                       outgoing_cltv_value: 42,
+                       cltv_expiry_height: 42,
                        payment_metadata: None,
                        keysend_preimage: None,
                        payment_data: Some(msgs::FinalOnionHopData {
@@ -12174,7 +12174,7 @@ mod tests {
                let current_height: u32 = node[0].node.best_block.read().unwrap().height();
                let result = create_recv_pending_htlc_info(msgs::InboundOnionPayload::Receive {
                        amt_msat: 100,
-                       outgoing_cltv_value: 22,
+                       cltv_expiry_height: 22,
                        payment_metadata: None,
                        keysend_preimage: None,
                        payment_data: Some(msgs::FinalOnionHopData {
index 2e56e2fc2c0c95d2e3cb3380d1f8b50138404ead..574f1b7cf05f8e8d3891e7eefed95e96135dc95d 100644 (file)
@@ -1707,7 +1707,7 @@ mod fuzzy_internal_msgs {
                        keysend_preimage: Option<PaymentPreimage>,
                        custom_tlvs: Vec<(u64, Vec<u8>)>,
                        amt_msat: u64,
-                       outgoing_cltv_value: u32,
+                       cltv_expiry_height: u32,
                },
                BlindedForward {
                        short_channel_id: u64,
@@ -2433,7 +2433,7 @@ impl<NS: Deref> ReadableArgs<(Option<PublicKey>, &NS)> for InboundOnionPayload w
                                payment_metadata: payment_metadata.map(|w| w.0),
                                keysend_preimage,
                                amt_msat: amt.ok_or(DecodeError::InvalidValue)?,
-                               outgoing_cltv_value: cltv_value.ok_or(DecodeError::InvalidValue)?,
+                               cltv_expiry_height: cltv_value.ok_or(DecodeError::InvalidValue)?,
                                custom_tlvs,
                        })
                }
@@ -4030,10 +4030,10 @@ mod tests {
                let node_signer = test_utils::TestKeysInterface::new(&[42; 32], Network::Testnet);
                let inbound_msg = ReadableArgs::read(&mut Cursor::new(&target_value[..]), (None, &&node_signer)).unwrap();
                if let msgs::InboundOnionPayload::Receive {
-                       payment_data: None, amt_msat, outgoing_cltv_value, ..
+                       payment_data: None, amt_msat, cltv_expiry_height, ..
                } = inbound_msg {
                        assert_eq!(amt_msat, 0x0badf00d01020304);
-                       assert_eq!(outgoing_cltv_value, 0xffffffff);
+                       assert_eq!(cltv_expiry_height, 0xffffffff);
                } else { panic!(); }
        }
 
@@ -4062,14 +4062,14 @@ mod tests {
                                payment_secret,
                                total_msat: 0x1badca1f
                        }),
-                       amt_msat, outgoing_cltv_value,
+                       amt_msat, cltv_expiry_height,
                        payment_metadata: None,
                        keysend_preimage: None,
                        custom_tlvs,
                } = inbound_msg  {
                        assert_eq!(payment_secret, expected_payment_secret);
                        assert_eq!(amt_msat, 0x0badf00d01020304);
-                       assert_eq!(outgoing_cltv_value, 0xffffffff);
+                       assert_eq!(cltv_expiry_height, 0xffffffff);
                        assert_eq!(custom_tlvs, vec![]);
                } else { panic!(); }
        }
@@ -4133,7 +4133,7 @@ mod tests {
                        keysend_preimage: None,
                        custom_tlvs,
                        amt_msat,
-                       outgoing_cltv_value,
+                       cltv_expiry_height: outgoing_cltv_value,
                        ..
                } = inbound_msg {
                        assert_eq!(custom_tlvs, expected_custom_tlvs);
index b04b78f6554da564fac04f13b602195635d5f7a1..ae602fb1103d5dab77c24a9c26bc27518453d97b 100644 (file)
@@ -126,9 +126,9 @@ pub(super) fn create_recv_pending_htlc_info(
                payment_metadata, requires_blinded_error
        ) = match hop_data {
                msgs::InboundOnionPayload::Receive {
-                       payment_data, keysend_preimage, custom_tlvs, amt_msat, outgoing_cltv_value, payment_metadata, ..
+                       payment_data, keysend_preimage, custom_tlvs, amt_msat, cltv_expiry_height, payment_metadata, ..
                } =>
-                       (payment_data, keysend_preimage, custom_tlvs, amt_msat, outgoing_cltv_value, payment_metadata,
+                       (payment_data, keysend_preimage, custom_tlvs, amt_msat, cltv_expiry_height, payment_metadata,
                         false),
                msgs::InboundOnionPayload::BlindedReceive {
                        amt_msat, total_msat, outgoing_cltv_value, payment_secret, intro_node_blinding_point,