Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Date: Fri, 31 May 2002 14:31:59 -0700
From: Justin Erenkrantz <jerenkrantz@apache.org>
To: dev@httpd.apache.org
Subject: Re: cvs commit: httpd-2.0/modules/http http_protocol.c
Message-ID: <20020531143159.X19485@apache.org>
Mail-Followup-To: Justin Erenkrantz <jerenkrantz@apache.org>,
	dev@httpd.apache.org
References: <20020531142047.W19485@apache.org>
 <00a701c208e9$2f0c4290$5600000a@KOJ>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <00a701c208e9$2f0c4290$5600000a@KOJ>;
 from rbb@covalent.net on Fri, May 31, 2002 at 02:21:52PM -0700

On Fri, May 31, 2002 at 02:21:52PM -0700, Ryan Bloom wrote:
> Without this fix, the entire test suite fails, because the HTTP_IN
> filter is sending requests with 0 Content-Length to the
> CORE_INPUT_FILTER to read the body.  This means that every request times
> out after some timeout.  It has nothing to do with Jeff's problem,
> because EVERY test was taking forever.  I did run the test-suite, so if
> this breaks anything, there is no test for it.

As I said, I didn't see this problem.  I'm currently building an
updated tree to test again.  If I can reproduce it, I'll try to
fix it.  However, we need the BODY_NONE state in order to handle
trailers.  Your commit breaks trailers.

httpd-test has no tests for input filtering.  If I knew how to
get perl to send bogus requests, I would.  But, my perl-fu is
severely lacking.  -- justin