-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Keep-alive support #17
Comments
Having the same issue, even if the headers are set the tunnel connection is dropped on every request and the connection isn't persistent. |
+1 |
What version of Node.js? Big changes have landed in the core HTTP implementation since this was initially written. |
@mikeal Node v6.x |
@mikeal I had this issue with 4.x and 6.x |
Any progress on keep-alive? It still does not seem to work. |
This library doesn't support Keep-Alive, but https://github.com/mknj/node-keepalive-proxy-agent seems to support it! |
Unfortunately this is the library that is used by https://github.com/faye/faye, so I was hoping there would be a way to get keep alives working here. |
In case of multiple consecutives requests, keep-alive is a good perf boost.
It is enabled by default in request but it does not seems to be supported with a proxy.
Is there any limitation due to tunneling or can this be achieved ?
The text was updated successfully, but these errors were encountered: