From: Yann Ylavic <ylavic@apache.org>
Date: Fri, 12 Aug 2016 14:04:51 +0000 (+0000)
Subject: mod_proxy_fcgi: forcing proxy_conn->close here is useless, same is done
X-Git-Tag: 2.5.0-alpha~1285
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ca4e35a740906bab3f36d2d9dd8ff867fe9a3fe4;p=apache

mod_proxy_fcgi: forcing proxy_conn->close here is useless, same is done
either above in ap_proxy_determine_connection(), or below in
ap_proxy_release_connection().

[Reverted by r1757097]


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

diff --git a/modules/proxy/mod_proxy_fcgi.c b/modules/proxy/mod_proxy_fcgi.c
index 90b1c7d895..10109813a5 100644
--- a/modules/proxy/mod_proxy_fcgi.c
+++ b/modules/proxy/mod_proxy_fcgi.c
@@ -938,16 +938,6 @@ 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)