You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by yl...@apache.org on 2016/08/21 23:53:24 UTC

svn commit: r1757097 - /httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c

Author: ylavic
Date: Sun Aug 21 23:53:24 2016
New Revision: 1757097

URL: http://svn.apache.org/viewvc?rev=1757097&view=rev
Log:
mod_proxy_fcgi: revert r1756187.

rpluem: The default is disablereuse=off and the code you removed makes the
default disablereuse=on.

ylavic: so right..

Modified:
    httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c?rev=1757097&r1=1757096&r2=1757097&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c Sun Aug 21 23:53:24 2016
@@ -938,6 +938,16 @@ static int proxy_fcgi_handler(request_re
         goto cleanup;
     }
 
+    /* This scheme handler does not reuse connections by default, to
+     * avoid tying up a fastcgi that isn't expecting to work on 
+     * parallel requests.  But if the user went out of their way to
+     * type the default value of disablereuse=off, we'll allow it.
+     */  
+    backend->close = 1;
+    if (worker->s->disablereuse_set && !worker->s->disablereuse) { 
+        backend->close = 0;
+    }
+
     /* Step Two: Make the Connection */
     if (ap_proxy_check_connection(FCGI_SCHEME, backend, r->server, 0,
                                   PROXY_CHECK_CONN_EMPTY)