Use RouteParametersConfig to create new RouteParameters

Author: shaavan

lightning/src/ln/outbound_payment.rs

@@ -851,7 +851,7 @@ impl OutboundPayments {
 		SP: Fn(SendAlongPathArgs) -> Result<(), APIError>,
 	{
 		let payment_hash = invoice.payment_hash();
-		let max_total_routing_fee_msat;
+		let params_config;
 		let retry_strategy;
 		match self.pending_outbound_payments.lock().unwrap().entry(payment_id) {
 			hash_map::Entry::Occupied(entry) => match entry.get() {
@@ -860,14 +860,17 @@ impl OutboundPayments {
 				} => {
 					retry_strategy = *retry;
 					// If max_total_fee is present, update route_params_config with the specified fee.
-					// This supports the standard behavior during downgrades.					
-					let route_params_config = max_total_fee.map_or(*route_params_config, |fee_msat| route_params_config.with_max_total_routing_fee_msat(fee_msat));
-					max_total_routing_fee_msat = route_params_config.max_total_routing_fee_msat;
+					// This supports the standard behavior during downgrades.
+					let route_params_config = max_total_fee.map_or(
+						*route_params_config,
+						|fee_msat| route_params_config.with_max_total_routing_fee_msat(fee_msat),
+					);
+					params_config = route_params_config;
 					*entry.into_mut() = PendingOutboundPayment::InvoiceReceived {
 						payment_hash,
 						retry_strategy: *retry,
 						max_total_routing_fee_msat: *max_total_fee,
-						route_params_config: route_params_config,
+						route_params_config,
 					};
 				},
 				_ => return Err(Bolt12PaymentError::DuplicateInvoice),
@@ -883,11 +886,11 @@ impl OutboundPayments {
 		}
 
 		let mut route_params = RouteParameters::from_payment_params_and_value(
-			PaymentParameters::from_bolt12_invoice(&invoice), invoice.amount_msats()
+			PaymentParameters::from_bolt12_invoice(&invoice).with_user_config(params_config), invoice.amount_msats()
 		);
-		if let Some(max_fee_msat) = max_total_routing_fee_msat {
-			route_params.max_total_routing_fee_msat = Some(max_fee_msat);
-		}
+
+		params_config.max_total_routing_fee_msat.map(|fee_msat| route_params.max_total_routing_fee_msat = Some(fee_msat));
+
 		self.send_payment_for_bolt12_invoice_internal(
 			payment_id, payment_hash, None, route_params, retry_strategy, router, first_hops,
 			inflight_htlcs, entropy_source, node_signer, node_id_lookup, secp_ctx, best_block_height,

lightning/src/routing/router.rs

@@ -875,6 +875,16 @@ impl PaymentParameters {
 		}
 	}
 
+	/// Update the parameters with the configuration provided by user.
+	pub fn with_user_config(self, params_config: RouteParametersConfig) -> Self {
+		Self {
+			max_total_cltv_expiry_delta: params_config.max_total_cltv_expiry_delta,
+			max_path_count: params_config.max_path_count,
+			max_channel_saturation_power_of_half: params_config.max_channel_saturation_power_of_half,
+			.. self
+		}
+	}
+
 	/// Includes the payee's features. Errors if the parameters were not initialized with
 	/// [`PaymentParameters::from_bolt12_invoice`].
 	///