]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: backend: don't use url_param_name as a hint for BE_LB_ALGO_PH
authorWilly Tarreau <w@1wt.eu>
Mon, 14 Jan 2019 14:17:46 +0000 (15:17 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 14 Jan 2019 18:33:17 +0000 (19:33 +0100)
commit089eaa0ba73913187e93d52c3ea34faa01fd8f9c
treef7fb6b124a3b92e7b2395f44292159521d4d5b54
parent9e7547740cc2d0a6851de8ca9ac57488bdbb8bf2
BUG/MINOR: backend: don't use url_param_name as a hint for BE_LB_ALGO_PH

At a few places in the code we used to rely on this variable to guess
what LB algo was in place. This is wrong because if the defaults section
presets "balance url_param foo" and a backend uses "balance roundrobin",
these locations will still see this url_param_name set and consider it.
The harm is limited, as this only causes the beginning of the request
body to be buffered. And in general this is a bad practice which prevents
us from cleaning the lbprm stuff. Let's explicitly check the LB algo
instead.

This may be backported to all currently maintained versions.
src/proto_http.c
src/proto_htx.c