Skip to content

Commit 6d0ac5d

Browse files
wtarreaua-denoyelle
authored andcommitted
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. (cherry picked from commit d8194fa) Signed-off-by: Amaury Denoyelle <[email protected]>
1 parent 5590ada commit 6d0ac5d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/cfgparse.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3053,7 +3053,7 @@ int check_config_validity()
30533053
ha_warning("'%s' will be ignored for %s '%s' (requires 'option httpchk').\n",
30543054
"send-state", proxy_type_str(curproxy), curproxy->id);
30553055
err_code |= ERR_WARN;
3056-
curproxy->options &= ~PR_O2_CHK_SNDST;
3056+
curproxy->options2 &= ~PR_O2_CHK_SNDST;
30573057
}
30583058
}
30593059

0 commit comments

Comments
 (0)