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

Amos Jeffries squid3 at treenet.co.nz
Thu Apr 9 01:07:00 UTC 2015


On 9/04/2015 3:12 a.m., Tsantilas Christos wrote:
> 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

+1. Please apply ASAP.

Amos



More information about the squid-dev mailing list