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 c793d99f407a3556a30f79a48469bb1b2ba93b83..f19a69b3645c6a1a6e14d222d9f98cb563060e41 100644 (file)
--- 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;
        }