httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Bloom" <...@covalent.net>
Subject RE: cvs commit: httpd-2.0 STATUS
Date Tue, 28 May 2002 22:38:38 GMT
> The problem I'm currently having is that the subreqs generated
> by the error page (HTTP_REQUEST_ENTITY_TOO_LARGE.html.var) is
> calling ap_discard_request_body() - which gets us our loop back
> into HTTP_IN (ap_http_filter).
>
> ....
>
> The idea is that if our status code is such that we're trying to
> avoid reading the body, we shouldn't actually read it.  We need
> the r->main trick as well because of subreqs (the .html.var file
> is a subreq handled by default_handler, so it will call discard_body
> as well on each subreq!).
> 
> How does this sound?  -- Justin

Sounds good, but I disagree with your STATUS code.  It is a 400, not a
413.  The request is completely invalid, not too large.  It would be too
large if we had set a limit on the size of requests, but that isn't the
problem.  The problem is that they have sent an invalid chunk.

Ryan



Mime
View raw message