]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: hlua: Don't set request analyzers on response channel for lua actions
authorChristopher Faulet <cfaulet@haproxy.com>
Fri, 26 Jul 2019 12:54:52 +0000 (14:54 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 29 Jul 2019 09:17:52 +0000 (11:17 +0200)
Setting some requests analyzers on the response channel was an old trick to be
sure to re-evaluate the request's analyers after the response's ones have been
called. It is no more necessary. In fact, this trick was removed in the version
1.8 and backported up to the version 1.6.

This patch must be backported to all versions since 1.6 to ease the backports of
fixes on the lua code.

src/hlua.c

index 9edef639804a04d573f580215144d8f10791f993..42d38491249a6d73f09dbf4d93c0b1f1ed9f0aac 100644 (file)
@@ -6232,11 +6232,8 @@ static enum act_return hlua_action(struct act_rule *rule, struct proxy *px,
                 * is detected on a response channel. This is useful
                 * only for actions targeted on the requests.
                 */
-               if (HLUA_IS_WAKERESWR(s->hlua)) {
+               if (HLUA_IS_WAKERESWR(s->hlua))
                        s->res.flags |= CF_WAKE_WRITE;
-                       if ((analyzer & (AN_REQ_INSPECT_FE|AN_REQ_HTTP_PROCESS_FE)))
-                               s->res.analysers |= analyzer;
-               }
                if (HLUA_IS_WAKEREQWR(s->hlua))
                        s->req.flags |= CF_WAKE_WRITE;
                /* We can quit the function without consistency check