Merge pull request #10439 from ziggie1984/bugfix/probing

Always add the payment address when probing an invoice

Author: yyforyongyu

docs/release-notes/release-notes-0.20.1.md

@@ -46,6 +46,11 @@
   issue](https://github.com/lightningnetwork/lnd/pull/10428) in LND which might
   happen when running postgres with a limited number of connections configured.
 
+* [Add missing payment address/secret when probing an 
+  invoice](https://github.com/lightningnetwork/lnd/pull/10439). This makes sure
+  the EstimateRouteFee API can probe Eclair and LDK nodes which enforce the
+  payment address/secret.
+
 # New Features
 
 ## Functional Enhancements

lnrpc/routerrpc/router_server.go

@@ -557,6 +557,7 @@ func (s *Server) probePaymentRequest(ctx context.Context, paymentRequest string,
 	// If the payment addresses is specified, then we'll also populate that
 	// now as well.
 	payReq.PaymentAddr.WhenSome(func(addr [32]byte) {
+		probeRequest.PaymentAddr = make([]byte, lntypes.HashSize)
 		copy(probeRequest.PaymentAddr, addr[:])
 	})
 
@@ -624,6 +625,10 @@ func (s *Server) probePaymentRequest(ctx context.Context, paymentRequest string,
 
 		// Copy the payment address if present.
 		if len(probeRequest.PaymentAddr) > 0 {
+			lspProbeRequest.PaymentAddr = make(
+				[]byte, lntypes.HashSize,
+			)
+
 			copy(
 				lspProbeRequest.PaymentAddr,
 				probeRequest.PaymentAddr,