-
Notifications
You must be signed in to change notification settings - Fork 283
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
Why does this warning message pop up? #20
Comments
It's an exception that is thrown because the connection timed out on a keep-alive connection. Whats missing is that the HTTP server sends a proper timeout and closes the connection after this timeout proactively instead of passively through the TCP read timeout. j-krueger is already on this. |
I still have this issue with latest vibed v0.7.12 (BTW you should update VibeVersionString). Lot's of "Handling of connection failed: Operating on closed TCPConnection" are generating. |
I still didn't find the root cause for this message in conjunction with Thanks for the version hint.. I completely missed that for this release. |
I start getting these "Handling of connection failed: Operating on closed TCPConnection." messages with a concurrency as low as 4 when testing with ab. Generally only 1 out of the 10,000 though. I can understand how these issues would be hard to track down. Loving Vibe BTW. :) ab -n 10000 -c 4 http://localhost:8080/ = ~ 1 message |
Interesting. I remember those but after fix was commited, have never reproduced ones, even on "-c 5000" (Have been doing perf tests recently). |
Right, I installed Vibe from GitHub this time and ran the same tests and it worked flawlessly. Previously I had used 'dub' and specified "vibe-d": ">=0.7.14" as a dependency in the packages.json. So it was my fault for not getting the latest code. Thanks! |
When a connection hasn't been used in a while, this message appears:
[7F3C31AC6F00:7F3C31ACAC80 WRN] Handling of connection failed: Operating on closed TCPConnection.
Is this really a warning? What is vibe trying to do here?
The text was updated successfully, but these errors were encountered: