cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH 2/7] pipelining: Fix connection handling under timeouts.

From: m brandenberg <mcbinc_at_panix.com>
Date: Thu, 13 Nov 2014 18:02:48 -0500 (EST)

On Tue, 11 Nov 2014, Daniel Stenberg wrote:

> ... as I still question that "bits.timedout" is necessary because I think we
> shouldn't add requests to a connection that is marked for close so
> connclose()
> should be enough. I thus think IsPipeliningPossible() should return FALSE if
> conn->bits.close is set to make sure it doesn't happen.

In the 1420 bug report, in my first attempt at a fix, I did exactly
that. The 'bits.close' flag was set rather frequently and the
result was the pipelining was mostly defeated. Second discovery
doing that was that the testing framework doesn't really test
whether pipelining occurs (it blew right over this).

--
Monty Brandenberg, Software Engineer                               MCB, Inc.
mcbinc_at_panix.com                                             P.O. Box 425292
mcbinc_at_pobox.com                                   Cambridge, MA  02142-0006
617.864.6907
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2014-11-14