if trampoline is used, prioritize mpp_splits that use the same trampoline

spesmilo / electrum

Electrum Bitcoin Wallet

https://electrum.org

MIT License

7.35k stars 3.06k forks source link

if trampoline is used, prioritize mpp_splits that use the same trampoline #7087

Open ecdsa opened 3 years ago

ecdsa commented 3 years ago

If trampoline is used, mpp_split should prioritize solutions that send HTLCs to the same trampoline. This will reduce fees, and also reduce the probability of failure.

ecdsa commented 3 years ago

@bitromortac

bitromortac commented 3 years ago

The issue with payment indexing was fixed in eclair, see https://github.com/ACINQ/eclair/issues/1723.

https://github.com/spesmilo/electrum/blob/9c1a51547a301e765b9b0f9935c6d940bb9d658e/electrum/lnworker.py#L1526-L1530 This should be no longer relevant and we could test sending via different trampoline nodes.