You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ni...@apache.org on 2008/05/13 22:21:21 UTC

svn commit: r655994 - /httpd/httpd/branches/2.2.x/STATUS

Author: niq
Date: Tue May 13 13:21:20 2008
New Revision: 655994

URL: http://svn.apache.org/viewvc?rev=655994&view=rev
Log:
vote, promote

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=655994&r1=655993&r2=655994&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Tue May 13 13:21:20 2008
@@ -90,6 +90,23 @@
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
+ * gen_test_char: add double-quote to the list of T_HTTP_TOKEN_STOP.
+   PR 9727 [Ville Skytt <ville.skytta iki.fi>]
+   Trunk version of patch:
+       http://svn.apache.org/viewvc?view=rev&revision=655714
+   Backport version for 2.2.x of patch:
+       Trunk version works
+  +1: fielding, rpluem, niq
+
+ * mod_proxy: Add in 'disablereuse' option for parity with similar
+   mod_jk functionality.
+   Trunk version of patch:
+       http://svn.apache.org/viewvc?view=rev&revision=627728
+   Backport version for 2.2.x of patch:
+       http://people.apache.org/~jim/patches/proxy-disablereuse-v2.txt
+   +1: jim, rpluem
+   niq: +1 with the proviso that there's still a typo in the docs in
+        proxy-disablereuse-v2.txt.  But it's fixed in r655986.
 
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
@@ -166,28 +183,6 @@
       http://people.apache.org/~jim/patches/proxy-ssl-44026-patch.txt
    +1: jim, rpluem
 
- * mod_proxy: Add in 'disablereuse' option for parity with similar
-   mod_jk functionality.
-   Trunk version of patch:
-       http://svn.apache.org/viewvc?view=rev&revision=627728
-   Backport version for 2.2.x of patch:
-       http://people.apache.org/~jim/patches/proxy-disablereuse-v2.txt
-   +1: jim, rpluem
-   niq: there's a typo in the docs and some trivial style matters.
-   More importantly, doesn't it require an MMN bump?
-   rpluem says: I agree with niq here that a minor bump is needed.
-   Further question: Why is the address not reusable (that means we
-   need to do a DNS query each time) when we only want to prevent that
-   this connection is kept alive?
-   jim: re-usable is a more "extensive" concept that just disabling
-        keepalives. What if, for example, the backend itself is
-        under round-robin DNS? In this case, we need to never
-        assume that the backend we've "just" connected to will
-        be the one we do the "next" time. Will update with mmn
-        bump. Not sure what style and typo changes are required.
-   jim: Latest rev of patch should address stated concerns :)
-   rpluem: Ok, convinced.
-
  * mod_proxy:  In the case that we fail to read the response line
    from the backend and if we are a reverse proxy request, shutdown
    the connection WITHOUT ANY response to trigger a retry by the client
@@ -255,14 +250,6 @@
       ?ids cannot, iirc, be signed - this is more a note to
       self ;) )
 
- * gen_test_char: add double-quote to the list of T_HTTP_TOKEN_STOP.
-   PR 9727 [Ville Skytt <ville.skytta iki.fi>]
-   Trunk version of patch:
-       http://svn.apache.org/viewvc?view=rev&revision=655714
-   Backport version for 2.2.x of patch:
-       Trunk version works
-  +1: fielding, rpluem
-
 * mod_ssl: Fix a memory leak with connections that have zlib compression
   turned on. PR 44975 [Joe Orton, Amund Elstad <Amund.Elstad ist.com>, 
   Dr Stephen Henson <steve openssl.org>]