Skip to content

Commit e48b882

Browse files
committed
Include a one-hop blinded path in Offer and Refund
While this doesn't add much privacy over not including any blinded paths, it allows us to exercise code for receiving on blinded paths.
1 parent c0aa1f6 commit e48b882

File tree

3 files changed

+24
-5
lines changed

3 files changed

+24
-5
lines changed

lightning/src/blinded_path/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ pub struct BlindedHop {
5757

5858
impl BlindedPath {
5959
/// Create a one-hop blinded path for a message.
60-
pub fn one_hop_for_message<ES: EntropySource, T: secp256k1::Signing + secp256k1::Verification>(
60+
pub fn one_hop_for_message<ES: EntropySource + ?Sized, T: secp256k1::Signing + secp256k1::Verification>(
6161
recipient_node_id: PublicKey, entropy_source: &ES, secp_ctx: &Secp256k1<T>
6262
) -> Result<Self, ()> {
6363
Self::new_for_message(&[recipient_node_id], entropy_source, secp_ctx)
@@ -68,7 +68,7 @@ impl BlindedPath {
6868
///
6969
/// Errors if less than one hop is provided or if `node_pk`(s) are invalid.
7070
// TODO: make all payloads the same size with padding + add dummy hops
71-
pub fn new_for_message<ES: EntropySource, T: secp256k1::Signing + secp256k1::Verification>(
71+
pub fn new_for_message<ES: EntropySource + ?Sized, T: secp256k1::Signing + secp256k1::Verification>(
7272
node_pks: &[PublicKey], entropy_source: &ES, secp_ctx: &Secp256k1<T>
7373
) -> Result<Self, ()> {
7474
if node_pks.is_empty() { return Err(()) }

lightning/src/ln/channelmanager.rs

Lines changed: 21 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ use bitcoin::secp256k1::{SecretKey,PublicKey};
3030
use bitcoin::secp256k1::Secp256k1;
3131
use bitcoin::{LockTime, secp256k1, Sequence};
3232

33+
use crate::blinded_path::BlindedPath;
3334
use crate::chain;
3435
use crate::chain::{Confirm, ChannelMonitorUpdateStatus, Watch, BestBlock};
3536
use crate::chain::chaininterface::{BroadcasterInterface, ConfirmationTarget, FeeEstimator, LowerBoundedFeeEstimator};
@@ -6824,6 +6825,9 @@ where
68246825
/// Creates an [`OfferBuilder`] such that the [`Offer`] it builds is recognized by the
68256826
/// [`ChannelManager`] when handling [`InvoiceRequest`] messages for the offer.
68266827
///
6828+
/// Uses a one-hop [`BlindedPath`] for the offer with [`ChannelManager::get_our_node_id`] as the
6829+
/// introduction node and a derived signing pubkey for recipient privacy.
6830+
///
68276831
/// [`Offer`]: crate::offers::offer::Offer
68286832
/// [`InvoiceRequest`]: crate::offers::invoice_request::InvoiceRequest
68296833
pub fn create_offer_builder(
@@ -6833,16 +6837,20 @@ where
68336837
let expanded_key = &self.inbound_payment_key;
68346838
let entropy = &*self.entropy_source;
68356839
let secp_ctx = &self.secp_ctx;
6840+
let path = self.create_one_hop_blinded_path();
68366841

6837-
// TODO: Set blinded paths
68386842
OfferBuilder::deriving_signing_pubkey(description, node_id, expanded_key, entropy, secp_ctx)
6843+
.path(path)
68396844
}
68406845

68416846
/// Creates a [`RefundBuilder`] such that the [`Refund`] it builds is recognized by the
68426847
/// [`ChannelManager`] when handling [`Bolt12Invoice`] messages for the refund.
68436848
///
68446849
/// The provided `payment_id` is used to ensure that only one invoice is paid for the refund.
68456850
///
6851+
/// Uses a one-hop [`BlindedPath`] for the refund with [`ChannelManager::get_our_node_id`] as
6852+
/// the introduction node and a derived payer id for sender privacy.
6853+
///
68466854
/// [`Refund`]: crate::offers::refund::Refund
68476855
/// [`Bolt12Invoice`]: crate::offers::invoice::Bolt12Invoice
68486856
pub fn create_refund_builder(
@@ -6852,11 +6860,12 @@ where
68526860
let expanded_key = &self.inbound_payment_key;
68536861
let entropy = &*self.entropy_source;
68546862
let secp_ctx = &self.secp_ctx;
6863+
let path = self.create_one_hop_blinded_path();
68556864

6856-
// TODO: Set blinded paths
68576865
let builder = RefundBuilder::deriving_payer_id(
68586866
description, node_id, expanded_key, entropy, secp_ctx, amount_msats, payment_id
6859-
)?;
6867+
)?.path(path);
6868+
68606869
self.pending_outbound_payments
68616870
.add_new_awaiting_invoice(payment_id, retry_strategy)
68626871
.map_err(|_| Bolt12SemanticError::DuplicatePaymentId)?;
@@ -6964,6 +6973,15 @@ where
69646973
inbound_payment::get_payment_preimage(payment_hash, payment_secret, &self.inbound_payment_key)
69656974
}
69666975

6976+
/// Creates a one-hop blinded path with [`ChannelManager::get_our_node_id`] as the introduction
6977+
/// node.
6978+
fn create_one_hop_blinded_path(&self) -> BlindedPath {
6979+
let entropy_source = self.entropy_source.deref();
6980+
let secp_ctx = &self.secp_ctx;
6981+
BlindedPath::one_hop_for_message(self.get_our_node_id(), entropy_source, secp_ctx).unwrap()
6982+
6983+
}
6984+
69676985
/// Gets a fake short channel id for use in receiving [phantom node payments]. These fake scids
69686986
/// are used when constructing the phantom invoice's route hints.
69696987
///

lightning/src/routing/router.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ pub trait Router {
9090
&self, payer: &PublicKey, route_params: &RouteParameters,
9191
first_hops: Option<&[&ChannelDetails]>, inflight_htlcs: InFlightHtlcs
9292
) -> Result<Route, LightningError>;
93+
9394
/// Finds a [`Route`] for a payment between the given `payer` and a payee.
9495
///
9596
/// The `payee` and the payment's value are given in [`RouteParameters::payment_params`]

0 commit comments

Comments
 (0)