BUG/MINOR: proxy: do not forget to validate quic-initial rules

In check_config_validity() and proxy_finalize() we check the consistency
of all rule sets, but the quic_initial rules were not placed there. This
currently has little to no impact, however we're going to use that to
also finalize certain debugging info so better call the function. This
can be backported to 3.1 (proxy_finalize is 3.4-only).

diff --git a/src/cfgparse.c b/src/cfgparse.c
index 8b99047830dbd9b4953d14d7e1346c61e95dc974..03578b144ad34e44ab09642fa9f94bca9baa8589 100644 (file)
--- a/src/cfgparse.c
+++ b/src/cfgparse.c
@@ -2377,6 +2377,9 @@ int check_config_validity()
                cfgerr += check_action_rules(&defpx->http_req_rules, defpx, &err_code);
                cfgerr += check_action_rules(&defpx->http_res_rules, defpx, &err_code);
                cfgerr += check_action_rules(&defpx->http_after_res_rules, defpx, &err_code);
+#ifdef USE_QUIC
+               cfgerr += check_action_rules(&defpx->quic_init_rules, defpx, &err_code);
+#endif
 
                err = NULL;
                i = smp_resolve_args(defpx, &err);

diff --git a/src/proxy.c b/src/proxy.c
index bfef16ddda5362d0e61e03c037e2324cb36cd2dc..1b6c8e4dbba77a442829cb5988251661ffc3fea4 100644 (file)
--- a/src/proxy.c
+++ b/src/proxy.c
@@ -2152,6 +2152,9 @@ int proxy_finalize(struct proxy *px, int *err_code)
        cfgerr += check_action_rules(&px->http_req_rules, px, err_code);
        cfgerr += check_action_rules(&px->http_res_rules, px, err_code);
        cfgerr += check_action_rules(&px->http_after_res_rules, px, err_code);
+#ifdef USE_QUIC
+       cfgerr += check_action_rules(&px->quic_init_rules, px, err_code);
+#endif
 
        /* Warn is a switch-mode http is used on a TCP listener with servers but no backend */
        if (!px->defbe.name && LIST_ISEMPTY(&px->switching_rules) && px->srv) {