Skip to content

Conversation

@stevenroose
Copy link
Contributor

It was causing this error in electrum 4.0.9:


E/i | interface.[localhost:50001] | Exception in wrapper_func: RequestCorrupted('expected 2016 headers but only got 2017')
Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/electrum/util.py", line 1056, in wrapper
    return await func(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 481, in wrapper_func
    return await func(self, *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 505, in run
    await self.open_session(ssl_context)
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 649, in open_session
    await group.spawn(self.monitor_connection)
  File "/usr/lib/python3.9/site-packages/aiorpcx/curio.py", line 242, in __aexit__
    await self.join()
  File "/usr/lib/python3.9/site-packages/aiorpcx/curio.py", line 211, in join
    raise task.exception()
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 703, in run_fetch_blocks
    await self._process_header_at_tip()
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 720, in _process_header_at_tip
    await self.sync_until(height)
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 729, in sync_until
    could_connect, num_headers = await self.request_chunk(height, next_height)
  File "/usr/lib/python3.9/site-packages/electrum/interface.py", line 613, in request_chunk
    raise RequestCorrupted(f"expected {size} headers but only got {res['count']}")
electrum.interface.RequestCorrupted: expected 2016 headers but only got 2017

@stevenroose
Copy link
Contributor Author

The assertion failed without the fix.

@shesek
Copy link
Collaborator

shesek commented Mar 18, 2021

Thanks!

I wonder how I never ran into this, probably only happens if the wallet is in some unusual state?

@shesek shesek merged commit 4844fe1 into bwt-dev:master Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants