You are viewing a plain text version of this content. The canonical link for it is here.

Posted to cvs@httpd.apache.org by el...@apache.org on 2016/07/27 13:48:26 UTC

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

Author: elukey
Date: Wed Jul 27 13:48:25 2016
New Revision: 1754272

URL: http://svn.apache.org/viewvc?rev=1754272&view=rev
Log:
Vote for r1751970

Tested manually and verified httpd's mod_proxy_fcgi logs with various request types 
(with/without query string, long query strings, etc..). 
My only doubt was related to the '\0' terminator placed in the middle of the char array,
but after a chat with Eric on IRC I got a clear view of what happens (only the chars up
to the \0 terminator are placed in FCGI headers later on). 
Last but not the least, in the bugzilla ticket the original requestor tested the change
and stated that solved his problem. Looks good to me for backport!


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=1754272&r1=1754271&r2=1754272&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Wed Jul 27 13:48:25 2016
@@ -180,7 +180,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
      string showing up in SCRIPT_FILENAME. PR59815
      trunk patch:  http://svn.apache.org/r1751970
      2.4.x patch: trunk works + CHANGES
-     +1 covener
+     +1 covener, elukey
 
   *) CMake: fix various issues for Windows/Visual Studio build environments.
      PR59685.