Skip to content

Commit 392bc6d

Browse files
committed
Decrypt Trampoline onions
In this commit, we start decrypting the inner onion we receive, and lay some groundwork towards handling the subsequent routing or payment receipt.
1 parent 4a10d2a commit 392bc6d

File tree

4 files changed

+459
-32
lines changed

4 files changed

+459
-32
lines changed

lightning/src/ln/blinded_payment_tests.rs

+145
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ use crate::util::test_utils;
3939
use lightning_invoice::RawBolt11Invoice;
4040
use types::features::Features;
4141
use crate::blinded_path::BlindedHop;
42+
use crate::routing::gossip::NodeId;
4243

4344
pub fn blinded_payment_path(
4445
payment_secret: PaymentSecret, intro_node_min_htlc: u64, intro_node_max_htlc: u64,
@@ -1755,3 +1756,147 @@ fn test_combined_trampoline_onion_creation_vectors() {
17551756
assert_eq!(htlc_msat, 150_156_000);
17561757
assert_eq!(htlc_cltv, 800_060);
17571758
}
1759+
1760+
#[test]
1761+
#[cfg(trampoline)]
1762+
fn test_trampoline_inbound_payment_decoding() {
1763+
let secp_ctx = Secp256k1::new();
1764+
let session_priv = secret_from_hex("0303030303030303030303030303030303030303030303030303030303030303");
1765+
1766+
let bob_secret = secret_from_hex("4242424242424242424242424242424242424242424242424242424242424242");
1767+
let bob_node_id = PublicKey::from_secret_key(&secp_ctx, &bob_secret);
1768+
let _bob_unblinded_tlvs = bytes_from_hex("011a0000000000000000000000000000000000000000000000000000020800000000000006c10a0800240000009627100c06000b69e505dc0e00fd023103123456");
1769+
let carol_secret = secret_from_hex("4343434343434343434343434343434343434343434343434343434343434343");
1770+
let carol_node_id = PublicKey::from_secret_key(&secp_ctx, &carol_secret);
1771+
let _carol_unblinded_tlvs = bytes_from_hex("020800000000000004510821031b84c5567b126440995d3ed5aaba0565d71e1834604819ff9c17f5e9d5dd078f0a0800300000006401f40c06000b69c105dc0e00");
1772+
let dave_secret = secret_from_hex("4444444444444444444444444444444444444444444444444444444444444444");
1773+
let dave_node_id = PublicKey::from_secret_key(&secp_ctx, &dave_secret);
1774+
let _dave_unblinded_tlvs = bytes_from_hex("01230000000000000000000000000000000000000000000000000000000000000000000000020800000000000002310a060090000000fa0c06000b699105dc0e00");
1775+
let eve_secret = secret_from_hex("4545454545454545454545454545454545454545454545454545454545454545");
1776+
let _eve_node_id = PublicKey::from_secret_key(&secp_ctx, &eve_secret);
1777+
let _eve_unblinded_tlvs = bytes_from_hex("011a00000000000000000000000000000000000000000000000000000604deadbeef0c06000b690105dc0e0f020000000000000000000000000000fdffff0206c1");
1778+
1779+
let path = Path {
1780+
hops: vec![
1781+
// Bob
1782+
RouteHop {
1783+
pubkey: bob_node_id,
1784+
node_features: NodeFeatures::empty(),
1785+
short_channel_id: 0,
1786+
channel_features: ChannelFeatures::empty(),
1787+
fee_msat: 0,
1788+
cltv_expiry_delta: 0,
1789+
maybe_announced_channel: false,
1790+
},
1791+
1792+
// Carol
1793+
RouteHop {
1794+
pubkey: carol_node_id,
1795+
node_features: NodeFeatures::empty(),
1796+
short_channel_id: (572330 << 40) + (42 << 16) + 2821,
1797+
channel_features: ChannelFeatures::empty(),
1798+
fee_msat: 150_153_000,
1799+
cltv_expiry_delta: 0,
1800+
maybe_announced_channel: false,
1801+
},
1802+
],
1803+
blinded_tail: Some(BlindedTail {
1804+
trampoline_hops: vec![
1805+
// Carol's pubkey
1806+
TrampolineHop {
1807+
pubkey: carol_node_id,
1808+
node_features: Features::empty(),
1809+
fee_msat: 2_500,
1810+
cltv_expiry_delta: 24,
1811+
},
1812+
// Dave's pubkey (the intro node needs to be duplicated)
1813+
TrampolineHop {
1814+
pubkey: dave_node_id,
1815+
node_features: Features::empty(),
1816+
fee_msat: 150_500, // incorporate both base and proportional fee
1817+
cltv_expiry_delta: 36,
1818+
}
1819+
],
1820+
hops: vec![
1821+
// Dave's blinded node id
1822+
BlindedHop {
1823+
blinded_node_id: pubkey_from_hex("0295d40514096a8be54859e7dfe947b376eaafea8afe5cb4eb2c13ff857ed0b4be"),
1824+
encrypted_payload: bytes_from_hex("0ccf3c8a58deaa603f657ee2a5ed9d604eb5c8ca1e5f801989afa8f3ea6d789bbdde2c7e7a1ef9ca8c38d2c54760febad8446d3f273ddb537569ef56613846ccd3aba78a"),
1825+
},
1826+
// Eve's blinded node id
1827+
BlindedHop {
1828+
blinded_node_id: pubkey_from_hex("020e2dbadcc2005e859819ddebbe88a834ae8a6d2b049233c07335f15cd1dc5f22"),
1829+
encrypted_payload: bytes_from_hex("bcd747394fbd4d99588da075a623316e15a576df5bc785cccc7cd6ec7b398acce6faf520175f9ec920f2ef261cdb83dc28cc3a0eeb970107b3306489bf771ef5b1213bca811d345285405861d08a655b6c237fa247a8b4491beee20c878a60e9816492026d8feb9dafa84585b253978db6a0aa2945df5ef445c61e801fb82f43d5f00716baf9fc9b3de50bc22950a36bda8fc27bfb1242e5860c7e687438d4133e058770361a19b6c271a2a07788d34dccc27e39b9829b061a4d960eac4a2c2b0f4de506c24f9af3868c0aff6dda27281c"),
1830+
}
1831+
],
1832+
blinding_point: pubkey_from_hex("02988face71e92c345a068f740191fd8e53be14f0bb957ef730d3c5f76087b960e"),
1833+
excess_final_cltv_expiry_delta: 0,
1834+
final_value_msat: 150_000_000
1835+
})
1836+
};
1837+
1838+
let payment_secret = PaymentSecret(secret_from_hex("7494b65bc092b48a75465e43e29be807eb2cc535ce8aaba31012b8ff1ceac5da").secret_bytes());
1839+
1840+
let amt_msat = 150_000_001;
1841+
let cur_height = 800_001;
1842+
let recipient_onion_fields = RecipientOnionFields::secret_only(payment_secret);
1843+
let (bob_onion, _, _) = onion_utils::create_payment_onion(&secp_ctx, &path, &session_priv, amt_msat, &recipient_onion_fields, cur_height, &PaymentHash([0; 32]), &None, None, [0; 32]).unwrap();
1844+
1845+
struct TestEcdhSigner {
1846+
node_secret: SecretKey,
1847+
}
1848+
impl NodeSigner for TestEcdhSigner {
1849+
fn ecdh(
1850+
&self, _recipient: Recipient, other_key: &PublicKey, tweak: Option<&Scalar>,
1851+
) -> Result<SharedSecret, ()> {
1852+
let mut node_secret = self.node_secret.clone();
1853+
if let Some(tweak) = tweak {
1854+
node_secret = self.node_secret.mul_tweak(tweak).map_err(|_| ())?;
1855+
}
1856+
Ok(SharedSecret::new(other_key, &node_secret))
1857+
}
1858+
fn get_inbound_payment_key(&self) -> ExpandedKey { unreachable!() }
1859+
fn get_node_id(&self, _recipient: Recipient) -> Result<PublicKey, ()> { unreachable!() }
1860+
fn sign_invoice(
1861+
&self, _invoice: &RawBolt11Invoice, _recipient: Recipient,
1862+
) -> Result<RecoverableSignature, ()> { unreachable!() }
1863+
fn sign_bolt12_invoice(
1864+
&self, _invoice: &UnsignedBolt12Invoice,
1865+
) -> Result<schnorr::Signature, ()> { unreachable!() }
1866+
fn sign_gossip_message(&self, _msg: UnsignedGossipMessage) -> Result<Signature, ()> { unreachable!() }
1867+
}
1868+
let logger = test_utils::TestLogger::with_id("".to_owned());
1869+
1870+
let bob_update_add = update_add_msg(111_000, 747_501, None, bob_onion);
1871+
let bob_node_signer = TestEcdhSigner { node_secret: bob_secret };
1872+
1873+
let (bob_peeled_onion, next_packet_details_opt) = onion_payment::decode_incoming_update_add_htlc_onion(
1874+
&bob_update_add, &bob_node_signer, &logger, &secp_ctx
1875+
).unwrap_or_else(|_| panic!());
1876+
1877+
let (carol_packet_bytes, carol_hmac) = if let onion_utils::Hop::Forward {
1878+
next_hop_data: msgs::InboundOnionForwardPayload {..}, next_hop_hmac, new_packet_bytes, ..
1879+
} = bob_peeled_onion {
1880+
(new_packet_bytes, next_hop_hmac)
1881+
} else { panic!() };
1882+
1883+
let carol_packet_details = next_packet_details_opt.unwrap();
1884+
let carol_onion = msgs::OnionPacket {
1885+
version: 0,
1886+
public_key: carol_packet_details.next_packet_pubkey,
1887+
hop_data: carol_packet_bytes,
1888+
hmac: carol_hmac,
1889+
};
1890+
let carol_update_add = update_add_msg(carol_packet_details.outgoing_amt_msat, carol_packet_details.outgoing_cltv_value, None, carol_onion);
1891+
1892+
let carol_node_signer = TestEcdhSigner { node_secret: carol_secret };
1893+
let (carol_peeled_onion, _) = onion_payment::decode_incoming_update_add_htlc_onion(
1894+
&carol_update_add, &carol_node_signer, &logger, &secp_ctx
1895+
).unwrap_or_else(|_| panic!());
1896+
1897+
let _carol_trampoline_update_add = if let onion_utils::Hop::TrampolineForward { next_trampoline_hop_data, .. } = carol_peeled_onion {
1898+
assert_eq!(next_trampoline_hop_data.outgoing_node_id, NodeId::from_pubkey(&dave_node_id));
1899+
} else {
1900+
panic!();
1901+
};
1902+
}

lightning/src/ln/channelmanager.rs

+26-1
Original file line numberDiff line numberDiff line change
@@ -4479,11 +4479,36 @@ where
44794479
Err(InboundHTLCErr { err_code, err_data, msg }) => return_err!(msg, err_code, &err_data)
44804480
}
44814481
},
4482+
#[cfg(trampoline)]
4483+
onion_utils::Hop::TrampolineReceive { .. } => {
4484+
// OUR PAYMENT!
4485+
let current_height: u32 = self.best_block.read().unwrap().height;
4486+
match create_recv_pending_htlc_info(decoded_hop, shared_secret, msg.payment_hash,
4487+
msg.amount_msat, msg.cltv_expiry, None, allow_underpay, msg.skimmed_fee_msat,
4488+
current_height)
4489+
{
4490+
Ok(info) => {
4491+
// Note that we could obviously respond immediately with an update_fulfill_htlc
4492+
// message, however that would leak that we are the recipient of this payment, so
4493+
// instead we stay symmetric with the forwarding case, only responding (after a
4494+
// delay) once they've sent us a commitment_signed!
4495+
PendingHTLCStatus::Forward(info)
4496+
},
4497+
Err(InboundHTLCErr { err_code, err_data, msg }) => return_err!(msg, err_code, &err_data)
4498+
}
4499+
},
44824500
onion_utils::Hop::Forward { .. } | onion_utils::Hop::BlindedForward { .. } => {
44834501
match create_fwd_pending_htlc_info(msg, decoded_hop, shared_secret, next_packet_pubkey_opt) {
44844502
Ok(info) => PendingHTLCStatus::Forward(info),
44854503
Err(InboundHTLCErr { err_code, err_data, msg }) => return_err!(msg, err_code, &err_data)
44864504
}
4505+
},
4506+
#[cfg(trampoline)]
4507+
onion_utils::Hop::TrampolineForward { .. } | onion_utils::Hop::TrampolineBlindedForward { .. } => {
4508+
match create_fwd_pending_htlc_info(msg, decoded_hop, shared_secret, next_packet_pubkey_opt) {
4509+
Ok(info) => PendingHTLCStatus::Forward(info),
4510+
Err(InboundHTLCErr { err_code, err_data, msg }) => return_err!(msg, err_code, &err_data)
4511+
}
44874512
}
44884513
}
44894514
}
@@ -5895,7 +5920,7 @@ where
58955920
// of the onion.
58965921
failed_payment!(err_msg, err_code, sha256_of_onion.to_vec(), None);
58975922
},
5898-
Err(onion_utils::OnionDecodeErr::Relay { err_msg, err_code, shared_secret }) => {
5923+
Err(onion_utils::OnionDecodeErr::Relay { err_msg, err_code, shared_secret, .. }) => {
58995924
let phantom_shared_secret = shared_secret.secret_bytes();
59005925
failed_payment!(err_msg, err_code, Vec::new(), Some(phantom_shared_secret));
59015926
},

0 commit comments

Comments
 (0)