f: fix edge case only when trampoline is also receiver

Author: a-mpch

lightning/src/ln/blinded_payment_tests.rs

@@ -2958,6 +2958,6 @@ fn test_trampoline_forward_rejection() {
 		// Expect UnknownNextPeer error while we are unable to route forwarding Trampoline payments.
 		let payment_failed_conditions = PaymentFailedConditions::new()
 			.expected_htlc_error_data(LocalHTLCFailureReason::UnknownNextPeer, &[0; 0]);
-		expect_payment_failed_conditions(&nodes[0], payment_hash, true, payment_failed_conditions);
+		expect_payment_failed_conditions(&nodes[0], payment_hash, false, payment_failed_conditions);
 	}
 }

lightning/src/ln/onion_utils.rs

@@ -1373,17 +1373,20 @@ where
 			short_channel_id = route_hop.short_channel_id()
 		}
 
-		// If next hop is from a trampoline we are already inside the trampoline
-		// route. If we found a permanent failure inside the trampoline route, we
-		// fail the payment permanently.
-		let is_next_hop_from_trampoline =
+		// If next hop is from a trampoline and there is only one trampoline hop
+		// in the trampoline route, means that the trampoline hop is also a
+		// final non-blinded node.
+		let is_next_hop_trampoline =
 			matches!(next_hop, Some((_, (Some(ErrorHop::TrampolineHop(..)), _))));
+		if is_next_hop_trampoline {
+			is_from_final_non_blinded_node =
+				path.blinded_tail.as_ref().map_or(0, |bt| bt.trampoline_hops.len()) == 1
+		}
 
 		res = Some(FailureLearnings {
 			network_update,
 			short_channel_id,
-			payment_failed_permanently: error_code.is_permanent()
-				&& (is_from_final_non_blinded_node || is_next_hop_from_trampoline),
+			payment_failed_permanently: error_code.is_permanent() && is_from_final_non_blinded_node,
 			failed_within_blinded_path: false,
 		});