Mailing-List: contact cvs-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r962985 - /httpd/httpd/branches/2.2.x/STATUS
Date: Sun, 11 Jul 2010 05:40:27 -0000
To: cvs@httpd.apache.org
From: niq@apache.org
Message-Id: <20100711054027.9790F2388999@eris.apache.org>

Author: niq
Date: Sun Jul 11 05:40:27 2010
New Revision: 962985

URL: http://svn.apache.org/viewvc?rev=962985&view=rev
Log:
vote, comment, query

Modified:
    httpd/httpd/branches/2.2.x/STATUS

Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=962985&r1=962984&r2=962985&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Sun Jul 11 05:40:27 2010
@@ -147,6 +147,16 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
          mod_ldap directives under a virtualhost are sufficient to trigger this merge issue)
      +1: covener, wrowe, trawick
 
+  *) mod_proxy_ajp: Ensure that we only return a non fatal error if the request
+     is idempotent and if we did not sent any request body so far.
+      Trunk version of patch:
+         http://svn.apache.org/viewvc?rev=955966&view=rev
+      Backport version for 2.2.x of patch:
+         Trunk version of patch works
+      +1: rpluem, trawick, niq
+     niq asks: did anyone check whether a similar change might be in order
+     in mod_proxy_http?  Or any of the CGI implementations?
+
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
 
@@ -207,6 +217,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
     Trunk patch: http://svn.apache.org/viewvc?rev=930125&view=rev
     2.2.x patch: https://issues.apache.org/bugzilla/attachment.cgi?id=25153
     Submitted by: Daniel Ruggeri <DRuggeri primary.net>
+    +1: niq
 
   * mod_disk_cache: Decline the opportunity to cache if the response is
     a 206 Partial Content. This stops a reverse proxied partial response
@@ -214,6 +225,9 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
     Trunk patch: http://svn.apache.org/viewvc?rev=951222&view=rev
     2.2.x patch: http://people.apache.org/~minfrin/httpd-cache-partial-2.2.patch
     +1: minfrin
+    niq asks: I can see the logic of not cacheing partial responses,
+    but why should mod_disk_cache worry about them if mod_cache allows
+    them, as in the following proposal?
 
   *) mod_cache: Explicitly allow cache implementations to cache a 206 Partial
      Response if they so choose to do so. Previously an attempt to cache a 206
@@ -223,14 +237,6 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
      2.2.x Patch: http://people.apache.org/~minfrin/httpd-cache-partial2-2.2.patch
      +1: minfrin
 
-  *) mod_proxy_ajp: Ensure that we only return a non fatal error if the request
-     is idempotent and if we did not sent any request body so far.
-      Trunk version of patch:
-         http://svn.apache.org/viewvc?rev=955966&view=rev
-      Backport version for 2.2.x of patch:
-         Trunk version of patch works
-      +1: rpluem, trawick
-
   *) mod_rewrite: Allow to set environment variables using mod_rewrite without
      explicitely giving a value.
      Most modules only check presence of a variable, not the value, so it makes
@@ -241,7 +247,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
         http://svn.apache.org/viewvc?rev=960233&view=rev
      Backport version for 2.2.x of patch:
         http://people.apache.org/~rjung/patches/httpd-branch-2.2-mod_rewrite-env_var-emptyvalue.patch
-     +1: rjung
+     +1: rjung, niq
 
   *) mod_filter: enable filtering of errordocuments.  PR 48377
      http://svn.apache.org/viewvc?view=revision&revision=894041