Exa-Networks / exaproxy

Performant Content Modifying Non-Caching Proxy [stable - accepting patches for bug fixes only]
Other
145 stars 36 forks source link

Fix: handle passthrough connections in redirector #48

Closed oriolarcas closed 5 years ago

oriolarcas commented 5 years ago

Before this fix, the redirector workers could not handle passthrough connections and forced a 'close' action (exaproxy/reactor/redirector/worker.py).

In addition, even if they answer properly ('intercept', which creates a downloader), it arrives asynchronously. So the initial 'read_client' hook is not enabled because it may happen before a downloader is created and the initial data from the client is discarded (losing the HTTP request).

Please confirm if this fix could be avoided by other means, like proper configuration, or it is the expected behavior and should be fixed.


This change is Reviewable