ethereum / web3.py

A python interface for interacting with the Ethereum blockchain and ecosystem.
http://web3py.readthedocs.io
MIT License
4.97k stars 1.7k forks source link

Fix buffer issues with async IPC; add tests #3492

Closed fselmo closed 2 weeks ago

fselmo commented 2 weeks ago

What was wrong?

Closes #3485

How was it fixed?

Todo:

Cute Animal Picture

ezgif com-animated-gif-maker

raxhvl commented 2 weeks ago

I wonder if chunks can arrive in arbitrary order. Also this thread discusses possible solutions.

fselmo commented 2 weeks ago

@raxhvl thanks for reporting. I believe this will be a more straightforward approach that can give control to the user to configure the read buffer limit if needs to be even larger than the default of 20MB. This should resolve your issues and I added a test to test this limit.