BUG/MINOR: log/balance: detect if user tries to use unsupported algo

b61147fd ("MEDIUM: log/balance: merge tcp/http algo with log ones")
introduced some ambiguities, because while it shares some algos with the
ones from mode {tcp,http}, we forgot report an error when the user tries
to use an algorithm that is not available in this mode (as per the doc).

Because of that, haproxy would silently drop log messages during runtime.

To fix that, we ensure that algo is one of the supported ones during log
backend postparsing. If the algo is not supported, we raise an error.

This should be backported in 2.9 with b61147fd

diff --git a/src/log.c b/src/log.c
index a25a5cf..c1e4477 100644 (file)
--- a/src/log.c
+++ b/src/log.c
@@ -921,6 +921,7 @@ static void log_backend_srv_down(struct server *srv)
 static int _postcheck_log_backend_compat(struct proxy *be)
 {
        int err_code = ERR_NONE;
+       int balance_algo = (be->lbprm.algo & BE_LB_ALGO);
 
        if (!LIST_ISEMPTY(&be->tcp_req.inspect_rules) ||
            !LIST_ISEMPTY(&be->tcp_req.l4_rules) ||
@@ -978,6 +979,13 @@ static int _postcheck_log_backend_compat(struct proxy *be)
                err_code |= ERR_WARN;
                free_server_rules(&be->server_rules);
        }
+       if (balance_algo != BE_LB_ALGO_RR &&
+           balance_algo != BE_LB_ALGO_RND &&
+           balance_algo != BE_LB_ALGO_LS &&
+           balance_algo != BE_LB_ALGO_LH) {
+               ha_alert("in %s '%s': \"balance\" only supports 'roundrobin', 'random', 'sticky' and 'log-hash'.\n", proxy_type_str(be), be->id);
+               err_code |= ERR_ALERT | ERR_FATAL;
+       }
        return err_code;
 }