From f3f4aa0266809c729ca98171a34de8c9bdd2c9f3 Mon Sep 17 00:00:00 2001 From: Tim Duesterhus Date: Fri, 3 Jul 2020 13:43:42 +0200 Subject: [PATCH] BUG/MINOR: http_act: don't check capture id in backend (2) Please refer to commit 19a69b3740702ce5503a063e9dfbcea5b9187d27 for all the details. This follow up commit fixes the `http-response capture` case, the previous one only fixed the `http-request capture` one. The documentation was already updated and the change to `check_http_res_capture` is identical to the `check_http_req_capture` change. This patch must be backported together with 19a69b3740702ce5503a063e9dfbcea5b9187d27. Most likely this is 1.6+. --- src/http_act.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/http_act.c b/src/http_act.c index 1c7a1d4e6..2eac12549 100644 --- a/src/http_act.c +++ b/src/http_act.c @@ -723,7 +723,10 @@ static int check_http_res_capture(struct act_rule *rule, struct proxy *px, char if (rule->action_ptr != http_action_res_capture_by_id) return 1; - if (rule->arg.capid.idx >= px->nb_rsp_cap) { + /* capture slots can only be declared in frontends, so we can't check their + * existence in backends at configuration parsing step + */ + if (px->cap & PR_CAP_FE && rule->arg.capid.idx >= px->nb_rsp_cap) { memprintf(err, "unable to find capture id '%d' referenced by http-response capture rule", rule->arg.capid.idx); return 0;