feat: add support for custom expiry

proto/sensei.proto

@@ -292,6 +292,7 @@ message KeysendResponse {}
 message CreateInvoiceRequest {
     uint64 amt_msat = 1;
     string description = 2;
+    optional uint32 expiry = 3;
 }
 message CreateInvoiceResponse {
     string invoice = 1;

senseicore/src/node.rs

@@ -66,7 +66,9 @@ use lightning::util::config::UserConfig;
 use lightning::util::ser::ReadableArgs;
 use lightning_background_processor::BackgroundProcessor;
 use lightning_invoice::utils::DefaultRouter;
-use lightning_invoice::{payment, utils, Currency, Invoice, InvoiceDescription};
+use lightning_invoice::{
+    payment, utils, Currency, Invoice, InvoiceDescription, DEFAULT_EXPIRY_TIME,
+};
 use lightning_net_tokio::SocketDescriptor;
 use lightning_rapid_gossip_sync::RapidGossipSync;
 use macaroon::Macaroon;
@@ -1185,21 +1187,29 @@ impl LightningNode {
         Ok(())
     }
 
-    pub async fn get_invoice(&self, amt_msat: u64, description: String) -> Result<Invoice, Error> {
+    pub async fn get_invoice(
+        &self,
+        amt_msat: u64,
+        description: String,
+        expiry: Option<u32>,
+    ) -> Result<Invoice, Error> {
         let currency = match self.config.network {
             Network::Bitcoin => Currency::Bitcoin,
             Network::Testnet => Currency::BitcoinTestnet,
             Network::Regtest => Currency::Regtest,
             Network::Signet => Currency::Signet,
         };
 
+        let invoice_expiry_delta_secs: u32 =
+            expiry.unwrap_or_else(|| DEFAULT_EXPIRY_TIME.try_into().unwrap());
+
         let invoice = utils::create_invoice_from_channelmanager(
             &self.channel_manager,
             self.keys_manager.clone(),
             currency,
             Some(amt_msat),
             description.clone(),
-            3600, // FIXME invoice_expiry_delta_secs
+            invoice_expiry_delta_secs,
         )?;
 
         let payment_hash = hex_utils::hex_str(&(*invoice.payment_hash()).into_inner());
@@ -1563,8 +1573,9 @@ impl LightningNode {
             NodeRequest::GetInvoice {
                 amt_msat,
                 description,
+                expiry,
             } => {
-                let invoice = self.get_invoice(amt_msat, description).await?;
+                let invoice = self.get_invoice(amt_msat, description, expiry).await?;
                 let invoice_str = format!("{}", invoice);
                 Ok(NodeResponse::GetInvoice {
                     invoice: invoice_str,

senseicore/src/services/node.rs

@@ -203,6 +203,7 @@ pub enum NodeRequest {
     GetInvoice {
         amt_msat: u64,
         description: String,
+        expiry: Option<u32>,
     },
     LabelPayment {
         label: String,

src/cli.rs

@@ -311,6 +311,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
                             let request = tonic::Request::new(CreateInvoiceRequest {
                                 amt_msat,
                                 description: String::from(""),
+                                expiry: Some(3600),
                             });
                             let response = client.create_invoice(request).await?;
                             println!("{:?}", response.into_inner());

src/grpc/adaptor.rs

@@ -352,6 +352,7 @@ impl From<CreateInvoiceRequest> for NodeRequest {
         NodeRequest::GetInvoice {
             amt_msat: req.amt_msat,
             description: req.description,
+            expiry: req.expiry,
         }
     }
 }

src/http/node.rs

@@ -31,13 +31,15 @@ use super::utils::get_macaroon_hex_str_from_cookies_or_header;
 pub struct GetInvoiceParams {
     pub amt_msat: u64,
     pub description: String,
+    pub expiry: Option<u32>,
 }
 
 impl From<GetInvoiceParams> for NodeRequest {
     fn from(params: GetInvoiceParams) -> Self {
         Self::GetInvoice {
             amt_msat: params.amt_msat,
             description: params.description,
+            expiry: params.expiry,
         }
     }
 }