Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Date: Thu, 6 Jun 2002 22:24:50 -0400 (EDT)
From: Ryan Bloom <rbb@ntrnet.net>
To: dev@httpd.apache.org
Subject: Re: ap_discard_request_body
In-Reply-To: <20020606183739.I8224@lyra.org>
Message-ID: <Pine.LNX.4.21.0206062223410.4163-100000@shell.ntrnet.net>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII

On Thu, 6 Jun 2002, Greg Stein wrote:

> On Thu, Jun 06, 2002 at 05:10:58PM -0700, Brian Pane wrote:
> > Ryan Bloom wrote:
> > 
> > >For the limits stuff, we need to decide if we want to return 413 if the
> > >server doesn't care about the request.  For example, default_handler
> > >always just discards the request body, so is it an error if the user
> > >sends 100 Meg of data with a request that is just going to ignore it?
> > >Probably, and if that is the case, then ap_discard_request_body has to
> > >stay in all of the handlers.
> > 
> > I don't think this case has to be considered an error.  We'll still
> > be compliant with 2616 if we return a 200 and discard a 100MB post
> > body that's been sent to, say, a static page.
> 
> Right.
> 
> > Assuming there's nothing else that breaks, I really like the idea
> > of simplifying the handlers' responsibilities by saying that only
> > the default_handler has to worry about discarding the request
> > body.
> 
> Huh? If you say "the handlers' responsibilities", then (by definition) the
> default handler is NOT going to be run.
> 
> I think this is entirely up to the HTTP_IN filter. When a request
> terminates, that filter *knows* the body has not (yet) been read. It should
> just proceed to read it in and toss it.

Well, it won't be the HTTP_IN filter, because that filter will never be
called unless something in the core calls it, but I can commit the code to
remove this logic from the handlers tonight sometime.

Thanks for the feedback.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
550 Jean St
Oakland CA 94610
-------------------------------------------------------------------------------