From 5abdc760c99a0011607f2cc97e199ef6ce0e8486 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Tue, 11 Jun 2019 16:01:56 +0200 Subject: [PATCH] BUG/MINOR: http-rules: mention "deny_status" for "deny" in the error message The error message indicating an unknown keyword on an http-request rule doesn't mention the "deny_status" option which comes with the "deny" rule, this is particularly confusing. This can be backported to all versions supporting this option. --- src/http_rules.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/http_rules.c b/src/http_rules.c index ab01f264c..508920020 100644 --- a/src/http_rules.c +++ b/src/http_rules.c @@ -556,7 +556,8 @@ struct act_rule *parse_http_req_cond(const char **args, const char *file, int li rule->cond = cond; } else if (*args[cur_arg]) { - ha_alert("parsing [%s:%d]: 'http-request %s' expects 'realm' for 'auth' or" + ha_alert("parsing [%s:%d]: 'http-request %s' expects 'realm' for 'auth'," + " 'deny_status' for 'deny', or" " either 'if' or 'unless' followed by a condition but found '%s'.\n", file, linenum, args[0], args[cur_arg]); goto out_err;