httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rpl...@apache.org
Subject svn commit: r721803 - /httpd/httpd/branches/2.2.x/STATUS
Date Sun, 30 Nov 2008 12:02:34 GMT
Author: rpluem
Date: Sun Nov 30 04:02:34 2008
New Revision: 721803

URL: http://svn.apache.org/viewvc?rev=721803&view=rev
Log:
* Add a comment

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=721803&r1=721802&r2=721803&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Sun Nov 30 04:02:34 2008
@@ -126,6 +126,13 @@
       niq says: +1 to this because it fixes a bogus error message.
                 But I don't see how this change passes AP_FILTER_ERROR
                 anywhere as advertised above.
+      rpluem says: Sorry for being confused by your commit message. Is this
+                   a full +1 that would enable us to backport or is this a
+                   conditional one?
+                   To answer your question: The patch causes ap_invoke_handler
+                   to leave a status of AP_FILTER_ERROR as is and not to convert it
+                   to HTTP_INTERNAL_SERVER_ERROR. So AP_FILTER_ERROR is passed on
+                   to the code processing the status of ap_invoke_handler.
 
 PATCHES/ISSUES THAT ARE STALLED
 



Mime
View raw message