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 2007/10/15 20:42:42 UTC
svn commit: r584845 - /httpd/httpd/branches/2.2.x/STATUS

Author: niq
Date: Mon Oct 15 11:42:42 2007
New Revision: 584845

URL: http://svn.apache.org/viewvc?rev=584845&view=rev
Log:
Address several points people have made in STATUS
(including proposal updates)

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=584845&r1=584844&r2=584845&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Mon Oct 15 11:42:42 2007
@@ -183,11 +183,13 @@
      PR 13986 and PR 16139
      http://svn.apache.org/viewvc?view=rev&revision=579991 (code)
      http://svn.apache.org/viewvc?view=rev&revision=580502 (docs)
+     http://svn.apache.org/viewvc?view=rev&revision=584842 (docs)
      +1: niq, rpluem
      rpluem says: IMHO the DefaultType documentation is now somewhat confusing
      (e.g. the server must add, but sometimes is better not to) and should be
      rewritten.
      jim says: Is rpluem's concern addressed?
+     niq says: Yes, when we add r584842 to the proposal.
      
    * mpm_winnt: Fix null pointer dereference
      PR 42572
@@ -196,15 +198,19 @@
 
    * mod_proxy_http: Don't mangle proxied URLs
      PR 42592
-     http://svn.apache.org/viewvc?view=rev&revision=583002
+     http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_http.c?r1=582655&r2=583803&pathrev=583803
      +1: niq
      -1: jim: As per Roy, (forward) proxies should not check
               the URI. See r583466 and r583803.
+     niq: updated proposal to address Roy & Jim's points.
 
    * core: backport ap_send_interim_response API
      http://svn.apache.org/viewvc?view=rev&revision=582630
      http://svn.apache.org/viewvc?view=rev&revision=582652
      +1: niq
+     niq says: It has been suggested the function should move to http_filters.
+     I'm OK with that if others feel happier that way.  The function
+     declaration presumably remains in http_protocol.h, so no API change.
 
    * mod_proxy_http: Correctly forward unexpected interim (HTTP 1xx) responses
      PR 16518
@@ -212,6 +218,8 @@
      +1: niq
      rpluem says: There is a conflict with the documentation (I don't know why).
      Once fixed I am +1.
+     niq says: Do you mean my addition to the documentation included
+     in r582631, or is there something else that needs reconciling?
 
 PATCHES/ISSUES THAT ARE STALLED