]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MEDIUM: check: remove checkport checkaddr flag
authorWilliam Dauchy <wdauchy@gmail.com>
Wed, 3 Feb 2021 21:30:07 +0000 (22:30 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 4 Feb 2021 09:43:00 +0000 (10:43 +0100)
commit69f118d7b60217c9f2acf9fc88b109670ef65e32
treecf12a0e964d0b4a898df64f045d6e5bcf7183c0d
parent21ca3dfc3a64e6accdb91539960d9f2dac63cdbc
MEDIUM: check: remove checkport checkaddr flag

While trying to fix some consistency problem with the config file/cli
(e.g. check-port cli command does not set the flag), we realised
checkport flag was not necessarily needed. Indeed tcpcheck uses service
port as the last choice if check.port is zero. So we can assume if
check.port is zero, it means it was never set by the user, regardless if
it is by the cli or config file.  In the longterm this will avoid to
introduce a new consistency issue if we forget to set the flag.

in the same manner of checkport flag, we don't really need checkaddr
flag. We can assume if checkaddr is not set, it means it was never set
by the user or config.

Signed-off-by: William Dauchy <wdauchy@gmail.com>
include/haproxy/server-t.h
src/check.c
src/server.c