BUG/MINOR: cfgparse: remove the correct option on httpcheck send-state warning

In GH issue #2586 @Bbulatov reported a bug where the http-check
send-state flag is removed from options instead of options2 when
http-check is disabled. It only has an effect when this option is
set and http-check disabled, where it displays a warning indicating
this will be ignored. The option removed instead is srvtcpka when
this happens. It's likely that both options being so minor, nobody
ever faced it.

This can be backported to all versions.

diff --git a/src/cfgparse.c b/src/cfgparse.c
index f5cde502cb269890e0adf56afc810d2688e93866..701a3700369234b96374364ea0262037077438a9 100644 (file)
--- a/src/cfgparse.c
+++ b/src/cfgparse.c
@@ -3053,7 +3053,7 @@ init_proxies_list_stage1:
                                ha_warning("'%s' will be ignored for %s '%s' (requires 'option httpchk').\n",
                                           "send-state", proxy_type_str(curproxy), curproxy->id);
                                err_code |= ERR_WARN;
-                               curproxy->options &= ~PR_O2_CHK_SNDST;
+                               curproxy->options2 &= ~PR_O2_CHK_SNDST;
                        }
                }