From: Yann Ylavic <ylavic@apache.org>
Date: Sun, 21 Aug 2016 23:53:24 +0000 (+0000)
Subject: mod_proxy_fcgi: revert r1756187.
X-Git-Tag: 2.5.0-alpha~1219
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=45d8216e5fbda24330f19d376b0387ee591fbfc0;p=thirdparty%2Fapache%2Fhttpd.git

mod_proxy_fcgi: revert r1756187.

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

ylavic: so right..


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1757097 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/proxy/mod_proxy_fcgi.c b/modules/proxy/mod_proxy_fcgi.c
index 10109813a51..90b1c7d895c 100644
--- a/modules/proxy/mod_proxy_fcgi.c
+++ b/modules/proxy/mod_proxy_fcgi.c
@@ -938,6 +938,16 @@ static int proxy_fcgi_handler(request_rec *r, proxy_worker *worker,
         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)