tln invoices: more relaxed filtering of chans to include route hints for - electrum - Electrum Bitcoin wallet
HTML git clone https://git.parazyd.org/electrum
DIR Log
DIR Files
DIR Refs
DIR Submodules
---
DIR commit c034219c5a31d09bcc183c5662f855799f112bb4
DIR parent 1788e5c1c094390958ee91a74aa3f048ba1e9a0c
HTML Author: SomberNight <somber.night@protonmail.com>
Date: Mon, 11 May 2020 16:01:33 +0200
ln invoices: more relaxed filtering of chans to include route hints for
e.g. just because remote peer is temporarily offline, we might still want it
included in the invoice
Diffstat:
M electrum/lnworker.py | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
---
DIR diff --git a/electrum/lnworker.py b/electrum/lnworker.py
t@@ -1216,19 +1216,21 @@ class LNWallet(LNWorker):
"""calculate routing hints (BOLT-11 'r' field)"""
routing_hints = []
channels = list(self.channels.values())
+ random.shuffle(channels) # not sure this has any benefit but let's not leak channel order
scid_to_my_channels = {chan.short_channel_id: chan for chan in channels
if chan.short_channel_id is not None}
- ignore_min_htlc_value = False
if amount_sat:
amount_msat = 1000 * amount_sat
else:
# for no amt invoices, check if channel can receive at least 1 msat
amount_msat = 1
- ignore_min_htlc_value = True
# note: currently we add *all* our channels; but this might be a privacy leak?
for chan in channels:
- if not chan.can_receive(amount_msat=amount_msat, check_frozen=True,
- ignore_min_htlc_value=ignore_min_htlc_value):
+ # do minimal filtering of channels.
+ # we include channels that cannot *right now* receive (e.g. peer disconnected or balance insufficient)
+ if not (chan.is_open() and not chan.is_frozen_for_receiving()):
+ continue
+ if amount_msat > 1000 * chan.constraints.capacity:
continue
chan_id = chan.short_channel_id
assert isinstance(chan_id, bytes), chan_id