Date: Wed, 12 Nov 1997 12:46:28 -0800 (PST)
From: Dean Gaudet <dgaudet@arctic.org>
To: new-httpd@apache.org
Subject: Re: protocol/1399: MISE 4.0 POST, then 401 Unauth,
 then second POST with good uname/pwd,
 garbage data in logs and (sometimes) garbled request (fwd) 
In-Reply-To: <9711120656.aa01021@paris.ics.uci.edu>
Message-ID: <Pine.LNX.3.95dg3.971112124346.30534H-100000@twinlark.arctic.org>
Organization: Transmeta Corp.
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: new-httpd-owner@apache.org
Precedence: bulk
Reply-To: new-httpd@apache.org


On Wed, 12 Nov 1997, Roy T. Fielding wrote:

> >> See discard_request_body in http_protocol.c. ... unfortunately it looks to
> >> only be called by default_handler() and not die(). 
> 
> Yep, I mentioned this when I made the original patch -- mod_cgi fails
> to do the right thing.  However, I didn't think about using die() to
> do it for us.  All it would require is the addition of
> 
>    dummy = discard_request_body(r);
> 
> which is safe to call even if there is no body or it has already been read.

Jim can you put this in "Open issues"?  We need to ensure all of our
content handlers do it.

> >I thought that Apache doesn't keep alive any error messages that came
> >from a request with an entity, for this reason. Maybe I'm remembering
> >incorrectly.
> 
> That changed around 1.2b8 or so.

Right, mea culpa.  That was when I was studying the performance for the
w3c pipelining paper ... and noticed that we dealt atrociously with a
pipeline full of GET If-Modified-Since requests.  Or something like that. 

Dean