-
Notifications
You must be signed in to change notification settings - Fork 295
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
net::ERR_INCOMPLETE_CHUNKED_ENCODING in node v8.0 + chrome 58.0.3029.110 #210
Comments
Here's a minimal repro: https://gist.github.com/egoist/55fed736c70eeb74a238a738f9f5ed13 |
From what I can tell, node is killing the request due to a timeout. Edit: I think it might be this: nodejs/node#2534. So there is now a 5s keep-alive timeout that can be set via It looks like keepAliveTimeout needs to be set on the server before any requests - as I understand Express middleware works on requests. const server = app.listen(4000, () => {
server.keepAliveTimeout = 0;
}); Using a heartbeat of 5s or less also works. |
Is there anything we can do per-request? Requiring server changes for a middleware is a bit annoying. Maybe we should just make the default heartbeat 4 seconds instead? |
Yeah, perhaps a lower default and a print a warning if the heartbeat is lower than keepAliveTimeout? |
I just started encountering the same issue after upgrading to Node 8. @brudil thanks for the fix, setting the server's keepAliveTimeout to 0 resolved the issue for me. |
* 1. response 302 on redirecting in router 2. heartbeat: 5000 for Node.js 8, via webpack-contrib/webpack-hot-middleware#210 (comment) * mistake * Create setup-dev-server.js * Create entry-server.js
Seems to work fine on node 8.1.1 |
I can confirm it's working with node 8.1.1 as well |
Also confirmed! 💃 Just gotta update to 8.1.1 and it's golden. |
Anyone know what changed in core to fix this for us? |
@glenjamin it's both nodejs/node#13435 + nodejs/node#13391 and the one @brudil pointed out in his reply its nodejs/node#2534. They're all pointing to the same issue. |
Cheers for the link, read through those and it looks like nodejs/node#13549 was the fix. Gonna call this issue closed |
upgrade node to latest version(
also fixed for me. |
The problem has been solved. thx |
* 1. response 302 on redirecting in router 2. heartbeat: 5000 for Node.js 8, via webpack-contrib/webpack-hot-middleware#210 (comment) * mistake * Create setup-dev-server.js * Create entry-server.js
* 1. response 302 on redirecting in router 2. heartbeat: 5000 for Node.js 8, via webpack-contrib/webpack-hot-middleware#210 (comment) * mistake * Create setup-dev-server.js * Create entry-server.js
Seeing this error
http://localhost:4000/__webpack_hmr net::ERR_INCOMPLETE_CHUNKED_ENCODING
in browser console after upgrading to node v8. As a result, code injection fails for certain updates.UPDATE:
Here's what I've tried and known to work
reference: egoist/poi#179
The text was updated successfully, but these errors were encountered: