Skip to content

Commit 1578d56

Browse files
committed
Stateless offer and refund builder utilities
Add utility functions to ChannelManager for creating OfferBuilder, InvoiceRequestBuilder, and RefundBuilder such that derived keys are used for the signing pubkey and payer id, respectively. This allows for stateless verification of any InvoiceRequest and Invoice messages. Later, blinded paths can be included in the returned builders. Also tracks future payments using the given PaymentId such that the corresponding Invoice is paid only once.
1 parent e3444c7 commit 1578d56

File tree

4 files changed

+109
-0
lines changed

4 files changed

+109
-0
lines changed

lightning/src/ln/channelmanager.rs

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,11 @@ use crate::ln::outbound_payment::{Bolt12PaymentError, OutboundPayments, PaymentA
5757
use crate::ln::wire::Encode;
5858
use crate::offers::invoice::{Bolt12Invoice, DEFAULT_RELATIVE_EXPIRY, DerivedSigningPubkey, InvoiceBuilder};
5959
use crate::offers::invoice_error::InvoiceError;
60+
use crate::offers::invoice_request::{DerivedPayerId, InvoiceRequestBuilder};
6061
use crate::offers::merkle::SignError;
62+
use crate::offers::offer::{DerivedMetadata, Offer, OfferBuilder};
6163
use crate::offers::parse::Bolt12SemanticError;
64+
use crate::offers::refund::RefundBuilder;
6265
use crate::onion_message::{OffersMessage, OffersMessageHandler};
6366
use crate::sign::{EntropySource, KeysManager, NodeSigner, Recipient, SignerProvider, WriteableEcdsaChannelSigner};
6467
use crate::util::config::{UserConfig, ChannelConfig, ChannelConfigUpdate};
@@ -6702,6 +6705,74 @@ where
67026705
}
67036706
}
67046707

6708+
/// Creates an [`OfferBuilder`] such that the [`Offer`] it builds is recognized by the
6709+
/// [`ChannelManager`] when handling [`InvoiceRequest`] messages for the offer.
6710+
///
6711+
/// [`Offer`]: crate::offers::offer::Offer
6712+
/// [`InvoiceRequest`]: crate::offers::invoice_request::InvoiceRequest
6713+
pub fn create_offer_builder(
6714+
&self, description: String
6715+
) -> OfferBuilder<DerivedMetadata, secp256k1::All> {
6716+
let node_id = self.get_our_node_id();
6717+
let expanded_key = &self.inbound_payment_key;
6718+
let entropy = &*self.entropy_source;
6719+
let secp_ctx = &self.secp_ctx;
6720+
6721+
// TODO: Set blinded paths
6722+
OfferBuilder::deriving_signing_pubkey(description, node_id, expanded_key, entropy, secp_ctx)
6723+
}
6724+
6725+
/// Creates a [`RefundBuilder`] such that the [`Refund`] it builds is recognized by the
6726+
/// [`ChannelManager`] when handling [`Bolt12Invoice`] messages for the refund.
6727+
///
6728+
/// The provided `payment_id` is used to ensure that only one invoice is paid for the refund.
6729+
///
6730+
/// [`Refund`]: crate::offers::refund::Refund
6731+
/// [`Bolt12Invoice`]: crate::offers::invoice::Bolt12Invoice
6732+
pub fn create_refund_builder(
6733+
&self, description: String, amount_msats: u64, payment_id: PaymentId, retry_strategy: Retry
6734+
) -> Result<RefundBuilder<secp256k1::All>, Bolt12SemanticError> {
6735+
let node_id = self.get_our_node_id();
6736+
let expanded_key = &self.inbound_payment_key;
6737+
let entropy = &*self.entropy_source;
6738+
let secp_ctx = &self.secp_ctx;
6739+
6740+
// TODO: Set blinded paths
6741+
let builder = RefundBuilder::deriving_payer_id(
6742+
description, node_id, expanded_key, entropy, secp_ctx, amount_msats, payment_id
6743+
)?;
6744+
self.pending_outbound_payments
6745+
.add_new_awaiting_invoice(payment_id, retry_strategy)
6746+
.map_err(|_| Bolt12SemanticError::DuplicatePaymentId)?;
6747+
6748+
Ok(builder)
6749+
}
6750+
6751+
/// Creates an [`InvoiceRequestBuilder`] such that the [`InvoiceRequest`] it builds is
6752+
/// recognized by the [`ChannelManager`] when handling [`Bolt12Invoice`] messages for the
6753+
/// request.
6754+
///
6755+
/// The provided `payment_id` is used to ensure that only one invoice is paid for the request.
6756+
///
6757+
/// [`InvoiceRequest`]: crate::offers::invoice_request::InvoiceRequest
6758+
/// [`Bolt12Invoice`]: crate::offers::invoice::Bolt12Invoice
6759+
pub fn create_invoice_request_builder<'a, 'b>(
6760+
&'b self, offer: &'a Offer, payment_id: PaymentId, retry_strategy: Retry
6761+
) -> Result<InvoiceRequestBuilder<'a, 'b, DerivedPayerId, secp256k1::All>, Bolt12SemanticError> {
6762+
let expanded_key = &self.inbound_payment_key;
6763+
let entropy = &*self.entropy_source;
6764+
let secp_ctx = &self.secp_ctx;
6765+
6766+
let builder = offer.request_invoice_deriving_payer_id(
6767+
expanded_key, entropy, secp_ctx, payment_id
6768+
)?;
6769+
self.pending_outbound_payments
6770+
.add_new_awaiting_invoice(payment_id, retry_strategy)
6771+
.map_err(|_| Bolt12SemanticError::DuplicatePaymentId)?;
6772+
6773+
Ok(builder)
6774+
}
6775+
67056776
/// Gets a payment secret and payment hash for use in an invoice given to a third party wishing
67066777
/// to pay us.
67076778
///

lightning/src/offers/offer.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
//! published as a QR code to be scanned by a customer. The customer uses the offer to request an
1414
//! invoice from the merchant to be paid.
1515
//!
16+
//! # Example
17+
//!
1618
//! ```
1719
//! extern crate bitcoin;
1820
//! extern crate core;
@@ -65,6 +67,14 @@
6567
//! # Ok(())
6668
//! # }
6769
//! ```
70+
//!
71+
//! # Note
72+
//!
73+
//! If constructing an [`Offer`] for use with a [`ChannelManager`], use
74+
//! [`ChannelManager::create_offer_builder`] instead of [`OfferBuilder::new`].
75+
//!
76+
//! [`ChannelManager`]: crate::ln::channelmanager::ChannelManager
77+
//! [`ChannelManager::create_offer_builder`]: crate::ln::channelmanager::ChannelManager::create_offer_builder
6878
6979
use bitcoin::blockdata::constants::ChainHash;
7080
use bitcoin::network::constants::Network;
@@ -132,6 +142,14 @@ impl<'a> OfferBuilder<'a, ExplicitMetadata, secp256k1::SignOnly> {
132142
/// while the offer is valid.
133143
///
134144
/// Use a different pubkey per offer to avoid correlating offers.
145+
///
146+
/// # Note
147+
///
148+
/// If constructing an [`Offer`] for use with a [`ChannelManager`], use
149+
/// [`ChannelManager::create_offer_builder`] instead of [`OfferBuilder::new`].
150+
///
151+
/// [`ChannelManager`]: crate::ln::channelmanager::ChannelManager
152+
/// [`ChannelManager::create_offer_builder`]: crate::ln::channelmanager::ChannelManager::create_offer_builder
135153
pub fn new(description: String, signing_pubkey: PublicKey) -> Self {
136154
OfferBuilder {
137155
offer: OfferContents {

lightning/src/offers/parse.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,8 @@ pub enum Bolt12SemanticError {
179179
MissingPayerMetadata,
180180
/// A payer id was expected but was missing.
181181
MissingPayerId,
182+
/// The payment id for a refund or request is already in use.
183+
DuplicatePaymentId,
182184
/// Blinded paths were expected but were missing.
183185
MissingPaths,
184186
/// The blinded payinfo given does not match the number of blinded path hops.

lightning/src/offers/refund.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@
1818
//! [`InvoiceRequest`]: crate::offers::invoice_request::InvoiceRequest
1919
//! [`Offer`]: crate::offers::offer::Offer
2020
//!
21+
//! # Example
22+
//!
2123
//! ```
2224
//! extern crate bitcoin;
2325
//! extern crate core;
@@ -70,6 +72,14 @@
7072
//! # Ok(())
7173
//! # }
7274
//! ```
75+
//!
76+
//! # Note
77+
//!
78+
//! If constructing a [`Refund`] for use with a [`ChannelManager`], use
79+
//! [`ChannelManager::create_refund_builder`] instead of [`RefundBuilder::new`].
80+
//!
81+
//! [`ChannelManager`]: crate::ln::channelmanager::ChannelManager
82+
//! [`ChannelManager::create_refund_builder`]: crate::ln::channelmanager::ChannelManager::create_refund_builder
7383
7484
use bitcoin::blockdata::constants::ChainHash;
7585
use bitcoin::network::constants::Network;
@@ -120,6 +130,14 @@ impl<'a> RefundBuilder<'a, secp256k1::SignOnly> {
120130
///
121131
/// Additionally, sets the required [`Refund::description`], [`Refund::payer_metadata`], and
122132
/// [`Refund::amount_msats`].
133+
///
134+
/// # Note
135+
///
136+
/// If constructing a [`Refund`] for use with a [`ChannelManager`], use
137+
/// [`ChannelManager::create_refund_builder`] instead of [`RefundBuilder::new`].
138+
///
139+
/// [`ChannelManager`]: crate::ln::channelmanager::ChannelManager
140+
/// [`ChannelManager::create_refund_builder`]: crate::ln::channelmanager::ChannelManager::create_refund_builder
123141
pub fn new(
124142
description: String, metadata: Vec<u8>, payer_id: PublicKey, amount_msats: u64
125143
) -> Result<Self, Bolt12SemanticError> {

0 commit comments

Comments
 (0)