]> git.ipfire.org Git - thirdparty/squid.git/commit
Author: Alex Rousskov <rousskov@measurement-factory.com>
authorAmos Jeffries <squid3@treenet.co.nz>
Sat, 22 May 2010 03:55:41 +0000 (15:55 +1200)
committerAmos Jeffries <squid3@treenet.co.nz>
Sat, 22 May 2010 03:55:41 +0000 (15:55 +1200)
commitb73a07d659fb5fb7bb8579888c6eb15f0098059f
tree2a58359de3962f04d3486007f393256902e631ad
parentde5c123a82d13950a95d6628efc6406816c8f64b
Author: Alex Rousskov <rousskov@measurement-factory.com>
Bug 2879: pt2: 3.0 regression in headers end finding

Consider the case when we received an empty (zero bytes) response.

The committed hack (bzr r9935) makes the "Invalid Response" warning misleading
because it adds CRLF to the empty response. The same hack makes the correct
error determination even more broken than it was (because the rest of the code
now sees content when none exited). It also mentions the wrong bug number.

The attached patch fixes the above and attempts to route empty response
processing to the right error (ERR_ZERO_SIZE_OBJECT).

TODO: Reconsider polluting cache.log with unlimited Invalid Response warnings,
at least in a forward proxy environment where the admin has no control over
responses.
src/http.cc