Legacy multi-trampoline mpp fails due to wrong total amount

[bitromortac]

I'm testing mpp payments with Electrum and we want to enable splitting across multiple trampoline forwarders.
Consider a splitting of (S: sender, R: receiver, T1: trampoline 1, T2: trampoline 2):

S ->- T1 ->- R (500/2000)
      T1 ->- R (500/2000)
S ->- T2 ->- R (1000/2000)

In this case, the sender tries to send 2000 sat to R, splitting accross T1 and T2. This already works for E2E trampoline payments, because the trampoline onions that arrive at the receiver have the information about the total amount from the trampoline onions.

In the legacy case, the trampolines T1, T2 carry out a payment to R and they can know about the total amount from their trampoline onion. Right now, T1 encodes a total amount of 1000 sat in the normal onion, causing a legacy receiver to fail the payment, because the total amount doesn't match the invoice amount.

I communicated with @t-bast, and the culprit lies in

eclair/eclair-core/src/main/scala/fr/acinq/eclair/payment/relay/NodeRelay.scala

Line 274 in fda3818

val payment = SendMultiPartPayment(payFsmAdapters, paymentSecret, payloadOut.outgoingNodeId, payloadOut.amountToForward, payloadOut.outgoingCltv, nodeParams.maxPaymentAttempts, routingHints, routeParams)

discarding the total amount from the trampoline onion and sending a different total amount.

Receiving trampoline onions will probably still take a while which is why I think it's important to support legacy mpp. I'll check whether I can find some time to fix this, if not anybody else wants to.