From: Willy Tarreau <w@1wt.eu>
Date: Thu, 20 Nov 2014 21:23:10 +0000 (+0100)
Subject: BUG/MINOR: stats: correctly set the request/response analysers
X-Git-Tag: v1.6-dev1~267
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5506e3f8b69927c1a381a268d5b0ca9907b363f5;p=thirdparty%2Fhaproxy.git

BUG/MINOR: stats: correctly set the request/response analysers

When enabling stats, response analysers were set on the request
analyser list, which 1) has no effect, and 2) means we don't have
the response analysers properly set.

In practice these response analysers are set when the connection
to the server or applet is established so we don't need/must not
set them here.

Fortunately this bug had no impact since the flags are distinct,
but it definitely is confusing.

It should be backported to 1.5.
---

diff --git a/src/proto_http.c b/src/proto_http.c
index c793d99f40..f19a69b364 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -4119,8 +4119,7 @@ int http_process_req_common(struct session *s, struct channel *req, int an_bit,
 			select_compression_request_header(s, req->buf);
 
 		/* enable the minimally required analyzers to handle keep-alive and compression on the HTTP response */
-		req->analysers = (req->analysers & AN_REQ_HTTP_BODY) |
-		                 AN_REQ_HTTP_XFER_BODY | AN_RES_WAIT_HTTP | AN_RES_HTTP_PROCESS_BE | AN_RES_HTTP_XFER_BODY;
+		req->analysers = (req->analysers & AN_REQ_HTTP_BODY) | AN_REQ_HTTP_XFER_BODY;
 		goto done;
 	}