svn commit: r1645877 - /httpd/httpd/branches/2.4.x/STATUS

[jk...@apache.org]

Author: jkaluza
Date: Tue Dec 16 08:19:26 2014
New Revision: 1645877

URL: http://svn.apache.org/r1645877
Log:
Propose r1644031, comment on ScriptLog patch (r1626978).

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

Modified: httpd/httpd/branches/2.4.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1645877&r1=1645876&r2=1645877&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Tue Dec 16 08:19:26 2014
@@ -192,13 +192,21 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
              The log_script_err() line may somewhere be expected in error_log.
              The new AH* in script_log could also cause parsing issues, but I'm
              fine with this personnaly.
+     jkaluza: It's long time I wrote it, but log_script_err calls
+              log_scripterror, which calls ap_log_rerror the same way as
+              original log_script_err did. Correct me, if I'm wrong, but that
+              should result in the log message being also in ErrorLog.
 
    * mod_proxy_fcgi: Ignore body data from backend for 304 responses. PR 57198.
      trunk patch: http://svn.apache.org/r1640495
+                  http://svn.apache.org/r1644031
      2.4.x patch: trunk works
-     +1 jkaluza, covener
+     +1 jkaluza
      ylavic: Shouldn't we handle more cases here to ignore the body?
              Please see my comment on dev@ as a reply to commit e-mail.
+     jkaluza: I've added also r1644031 as reaction to Yann's comment above.
+              Eric, I'm removing your +1, because it's based just on original
+              r1640495. See my reply on dev@ for more info.
 
    * mod_proxy: Add ap_proxy_define_match_worker() and use it for ProxyPassMatch
      and ProxyMatch section to distinguish between normal workers and workers