From: Justin Erenkrantz <jerenkrantz@apache.org>
Date: Fri, 7 Jun 2002 00:59:03 +0000 (+0000)
Subject: Clarify note on ap_discard_request_body().  If we want to change it,
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3767a822b31581723b0b114b06efdc258b42f78e;p=apache

Clarify note on ap_discard_request_body().  If we want to change it,
we should do it before .37 - hence its remaining as a showstopper.

Add note on the 304 showstopper (aka PR 9673 in BZ).


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@95555 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/STATUS b/STATUS
index f11327ca6e..bfed6a453e 100644
--- a/STATUS
+++ b/STATUS
@@ -1,5 +1,5 @@
 APACHE 2.0 STATUS:                                              -*-text-*-
-Last modified at [$Date: 2002/06/06 01:08:06 $]
+Last modified at [$Date: 2002/06/07 00:59:03 $]
 
 Release:
 
@@ -67,21 +67,29 @@ RELEASE SHOWSTOPPERS:
       BrianP says:  how about "ap_apachectl()"?
       Justin says to BrianP: "Eww."
 
-    * Chunked trailer support is broken.
+    * Potentially remove ap_discard_request_body()
       Message-ID: <20020531142047.W19485@apache.org>
-      Status: rbb committed a fix to http_protocol.c, but we still need to
-              figure out what to do with ap_discard_request_body.  Should
-              we still rearrange it like the patches below have it?  Or,
-              can we remove all callers of ap_discard_request_body and
-              only do it in ap_finalize_request_protocol?  (rbb hinted that
-              we can't do it in ap_finalize_request_protocol because of
-              potential 413 errors which would be detected too late.)
+                  <3CFFFA12.1040007@cnet.com>
+      Status: Can we remove all callers of ap_discard_request_body and
+              only do it in ap_finalize_request_protocol?  If we do it
+              in finalize_request_protocol, we may not catch 413 errors
+              until after the request is served.  Oops.  But, if we
+              didn't NEED the body in the first place, then it isn't
+              really a 413, is it?
+      Patches (may or may not be applicable):
                 For the core and various modules:
                   <20020602164041.N19485@apache.org>
                 For mod_dav:
                   <20020602161511.K19485@apache.org>
                   <20020602162838.L19485@apache.org>
 
+    * Returning invalid 304 errors when filters are present.  [PR 9673]
+      Message-ID: <Pine.GSO.4.44.0206061713320.3848-100000@lab26.lacfas.hec.ca>
+      Status: We should set r->no_local_copy, but where can we set it?
+              We must set it before the handlers are run so that the
+              appropriate handler can call ap_meets_conditions() before
+              generating the data.
+
 CURRENT VOTES:
 
     * apachectl should revert to just being an init script and