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)