httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From elu...@apache.org
Subject svn commit: r1754272 - /httpd/httpd/branches/2.4.x/STATUS
Date Wed, 27 Jul 2016 13:48:26 GMT
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.



Mime
View raw message