HttpStreamParser: Don't reuse sockets which receive unparsed data.
If HttpStreamParser has extra data left over after consuming a valid
HTTP/1.x response on a socket, it would still allow the socket to be
reused if it was connected and idle. Since we have no idea what the
data actually was, this just seems like a bad idea. This CL changes
that behavior, so such sockets are no longer considered reuseable.
This does have the downside of papering over server bugs, but it
still seems like the right way to go.
5 files changed