[squid-dev] [PATCH] Fix HttpStateData::readReply to retry reads from server

Tsantilas Christos chtsanti at users.sourceforge.net
Wed Apr 8 15:12:38 UTC 2015


Hi all,

This patch fixes HttpStateData::readReply to retry read from server in 
the case of EINPROGRESS, EAGAIN or similar errors

This bug mostly affects SSL bumped connections. The 
HttpStateData::readReply will not retry read from server in the case of 
an EINPROGRESS or similar comm errors and the connection will hang, 
until the timeout handler called.

The Comm::ReadNow method, used inside  HttpStateData::readReply, call 
ignoreErrno function to test if the comm error should be ignored and in 
this case return Comm::INPROGRESS value.
In this case we need to set flags.do_next_read to true to force 
HttpStateData::maybeReadVirginBody() method retry read.

This is a Measurement Factory project
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-HttpStateData_readReply-t1.patch
Type: text/x-patch
Size: 3860 bytes
Desc: not available
URL: <http://lists.squid-cache.org/pipermail/squid-dev/attachments/20150408/36437bc7/attachment.bin>


More information about the squid-dev mailing list