spesmilo / electrum
61%
master: 62%

Repo Added 10 Nov 2017 08:46AM UTC

Committed 28 Oct 2025 02:42PM UTC coverage: 61.35% (+0.009%) from 61.341%

Committed by

Commit Message

lnpeer: await init in main_loop

Because `LNPeer.initialized` was awaited in
`LNPeer._query_gossip()` instead of the main loop the other tasks got
spawned concurrently and each task on its own has to wait for the
initialization. In `LNPeer._send_own_gossip()` this was missing, instead
there is a fixed 10 sec sleep. If the connection was not initialized but
the 10 sec are exceeded `_send_own_gossip()` tries to send gossip and
causes this exception as the `LNTransport` is not ready:

```
  2.13 | E | lnpeer.Peer.[LNWallet, 0288fa27c0-bc1900c8] | Exception in main_loop: AttributeError("'LNTransport' object has no attribute 'sk'")
Traceback (most recent call last):
  File "/home/user/code/electrum-fork/electrum/util.py", line 1232, in wrapper
    return await func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnpeer.py", line 511, in wrapper_func
    return await func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnpeer.py", line 525, in main_loop
    async with self.taskgroup as group:
               ^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/env/lib/python3.14/site-packages/aiorpcx/curio.py", line 304, in __aexit__
    await self.join()
  File "/home/user/code/electrum-fork/electrum/util.py", line 1420, in join
    task.result()
    ~~~~~~~~~~~^^
  File "/home/user/code/electrum-fork/electrum/lnpeer.py", line 573, in _send_own_gossip
    self.send_node_announcement(alias, color)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnpeer.py", line 1830, in send_node_announcement
    self.transport.send_bytes(raw_msg)
    ~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lntransport.py", line 225, in send_bytes
    lc = aead_encrypt(self.sk, self.sn(), b'', l)
                      ^^^^^^^
AttributeError: 'LNTransport' object has no attribute 'sk'. D... (continued)

0 of 8 new or added lines in 1 file covered. (0.0%)

2 existing lines in 2 files now uncovered.

22900 of 37327 relevant lines covered (61.35%)

0.61 hits per line

37327 RELEVANT LINES 22900 COVERED LINES

0.61 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
46031474...	lnpeer_dont_send_gossip_before_init	lnpeer: await init in main_loop Because `LNPeer.initialized` was awaited in `LNPeer._query_gossip()` instead of the main loop the other tasks got spawned concurrently and each task on its own has to wait for the initialization. In `LNPeer._send_o...	Pull #10285	28 Oct 2025 02:48PM UTC	f321x	CirrusCI	61.35

spesmilo / electrum
61%
master: 62%

spesmilo / electrum 61% master: 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on lnpeer_dont_send_gossip_before_init

Recent builds

spesmilo / electrum
61%
master: 62%

README BADGES
x

CHANGE BRANCH
x