Return payment hash and secret in Bolt11ReceiveResponse

Before you would create an invoice and then need to decode it to get the
payment hash/secret to then match it to a PaymentReceived event. Now
we'll include those in the Bolt11ReceiveResponse to make it easier.

Author: benthecarman

e2e-tests/tests/e2e.rs

@@ -14,9 +14,10 @@ use e2e_tests::{
 	find_available_port, mine_and_sync, run_cli, run_cli_raw, setup_funded_channel,
 	wait_for_onchain_balance, LdkServerHandle, RabbitMqEventConsumer, TestBitcoind,
 };
-use hex_conservative::DisplayHex;
+use hex_conservative::{DisplayHex, FromHex};
 use ldk_node::bitcoin::hashes::{sha256, Hash};
 use ldk_node::lightning::ln::msgs::SocketAddress;
+use ldk_node::lightning_invoice::Bolt11Invoice;
 use ldk_server_client::ldk_server_protos::api::{
 	Bolt11ReceiveRequest, Bolt12ReceiveRequest, OnchainReceiveRequest,
 };
@@ -136,8 +137,14 @@ async fn test_cli_bolt11_receive() {
 	let server = LdkServerHandle::start(&bitcoind).await;
 
 	let output = run_cli(&server, &["bolt11-receive", "50000sat", "-d", "test"]);
-	let invoice = output["invoice"].as_str().unwrap();
-	assert!(invoice.starts_with("lnbcrt"), "Expected lnbcrt prefix, got: {}", invoice);
+	let invoice_str = output["invoice"].as_str().unwrap();
+	assert!(invoice_str.starts_with("lnbcrt"), "Expected lnbcrt prefix, got: {}", invoice_str);
+
+	let invoice: Bolt11Invoice = invoice_str.parse().unwrap();
+	let payment_hash = sha256::Hash::from_str(output["payment_hash"].as_str().unwrap()).unwrap();
+	assert_eq!(*invoice.payment_hash(), payment_hash);
+	let payment_secret = <[u8; 32]>::from_hex(output["payment_secret"].as_str().unwrap()).unwrap();
+	assert_eq!(invoice.payment_secret().0, payment_secret);
 }
 
 #[tokio::test]

ldk-server-protos/src/api.rs

@@ -173,6 +173,12 @@ pub struct Bolt11ReceiveResponse {
 	/// to the recipient.
 	#[prost(string, tag = "1")]
 	pub invoice: ::prost::alloc::string::String,
+	/// The hex-encoded 32-byte payment hash.
+	#[prost(string, tag = "2")]
+	pub payment_hash: ::prost::alloc::string::String,
+	/// The hex-encoded 32-byte payment secret.
+	#[prost(string, tag = "3")]
+	pub payment_secret: ::prost::alloc::string::String,
 }
 /// Return a BOLT11 payable invoice for a given payment hash.
 /// The inbound payment will NOT be automatically claimed upon arrival.

ldk-server-protos/src/proto/api.proto

@@ -146,6 +146,12 @@ message Bolt11ReceiveResponse {
   // With the details of the invoice, the sender has all the data necessary to send a payment
   // to the recipient.
   string invoice = 1;
+
+  // The hex-encoded 32-byte payment hash.
+  string payment_hash = 2;
+
+  // The hex-encoded 32-byte payment secret.
+  string payment_secret = 3;
 }
 
 // Return a BOLT11 payable invoice for a given payment hash.

ldk-server/src/api/bolt11_receive.rs

@@ -7,6 +7,7 @@
 // You may not use this file except in accordance with one or both of these
 // licenses.
 
+use hex::DisplayHex;
 use ldk_server_protos::api::{Bolt11ReceiveRequest, Bolt11ReceiveResponse};
 
 use crate::api::error::LdkServerError;
@@ -27,6 +28,9 @@ pub(crate) fn handle_bolt11_receive_request(
 			.receive_variable_amount(&description, request.expiry_secs)?,
 	};
 
-	let response = Bolt11ReceiveResponse { invoice: invoice.to_string() };
+	let payment_hash = invoice.payment_hash().0.to_lower_hex_string();
+	let payment_secret = invoice.payment_secret().0.to_lower_hex_string();
+	let response =
+		Bolt11ReceiveResponse { invoice: invoice.to_string(), payment_hash, payment_secret };
 	Ok(response)
 }