]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: sink: detect and warn when using "send-proxy" options with ring servers
authorAurelien DARRAGON <adarragon@haproxy.com>
Thu, 15 May 2025 13:54:13 +0000 (15:54 +0200)
committerAurelien DARRAGON <adarragon@haproxy.com>
Thu, 15 May 2025 14:18:31 +0000 (16:18 +0200)
commit098a5e5c0b9e6821c03945295dd1350e5d834eb3
treea3c6dabe5b3cd45596a07e1139decdf11ca03820
parent824bb93e18043876a0fa30eeb3de50ea0ccb4ea5
BUG/MINOR: sink: detect and warn when using "send-proxy" options with ring servers

using "send-proxy" or "send-proxy-v2" option on a ring server is not
relevant nor supported. Worse, on 2.4 it causes haproxy process to
crash as reported in GH #2965.

Let's be more explicit about the fact that this keyword is not supported
under "ring" context by ignoring the option and emitting a warning message
to inform the user about that.

Ideally, we should do the same for peers and log servers. The proper way
would be to check servers options during postparsing but we currently lack
proper cross-type server postparsing hooks. This will come later and thus
will give us a chance to perform the compatibilty checks for server
options depending on proxy type. But for now let's simply fix the "ring"
case since it is the only one that's known to cause a crash.

It may be backported to all stable versions.
src/sink.c