]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: acl: Fix type of log message when an acl is named 'or'
authorTim Duesterhus <tim@bastelstu.be>
Thu, 6 Feb 2020 21:04:03 +0000 (22:04 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 6 Feb 2020 21:16:07 +0000 (22:16 +0100)
The patch adding this check initially only issued a warning, instead of
being fatal. It was changed before committing. However when making this
change the type of the log message was not changed from `ha_warning` to
`ha-alert`. This patch makes this forgotten adjustment.

see 0cf811a5f941261176b67046dbc542d0479ff4a7
No backport needed. The initial patch was backported as a warning, thus
the log message type is correct.

src/cfgparse-listen.c
src/fcgi-app.c
src/flt_spoe.c

index 77a5d5b8fbc043dab8dac2bd29bf0b93f80334ba..6d9c1f52e6b000516ba27436c2280f1625641efa 100644 (file)
@@ -808,7 +808,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm)
                }
 
                if (strcasecmp(args[1], "or") == 0) {
-                       ha_warning("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
+                       ha_alert("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
                                   "logical disjunction within a condition.\n",
                                   file, linenum, args[1]);
                        err_code |= ERR_ALERT | ERR_FATAL;
index 7b28e3a9cfd9fb9bf2c5cb4f4a523408d20935d0..a9466a577031875434a4411958da30e06e2ae1b6 100644 (file)
@@ -888,7 +888,7 @@ static int cfg_parse_fcgi_app(const char *file, int linenum, char **args, int kw
                        goto out;
                }
                if (strcasecmp(args[1], "or") == 0) {
-                       ha_warning("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
+                       ha_alert("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
                                   "logical disjunction within a condition.\n",
                                   file, linenum, args[1]);
                        err_code |= ERR_ALERT | ERR_FATAL;
index 06c70d24fa730c4001e9f4583cd1065e0b63c2ce..4e448dd1124697375b3236382768947d5b2461fb 100644 (file)
@@ -3992,7 +3992,7 @@ cfg_parse_spoe_message(const char *file, int linenum, char **args, int kwm)
                        goto out;
                }
                if (strcasecmp(args[1], "or") == 0) {
-                       ha_warning("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
+                       ha_alert("parsing [%s:%d] : acl name '%s' will never match. 'or' is used to express a "
                                   "logical disjunction within a condition.\n",
                                   file, linenum, args[1]);
                        err_code |= ERR_ALERT | ERR_FATAL;