From: Christopher Faulet <cfaulet@haproxy.com>
Date: Fri, 13 Jan 2023 07:53:23 +0000 (+0100)
Subject: BUG/MINOR: promex: Don't forget to consume the request on error
X-Git-Tag: v2.8-dev2~47
X-Git-Url: http://www.kaiwu.me/postgresql/commit/static/gitweb.js?a=commitdiff_plain;h=c1b013bc61e165cd54fc3e4c538f95ba0c0104c6;p=haproxy.git

BUG/MINOR: promex: Don't forget to consume the request on error

When the promex applet triggers an error, for instance because the URI is
invalid, we must still take care to consume the request. Otherwise, the
error will be handled by HTTP analyzers as a server abort.

This patch must be backported as far as 2.0.
---

diff --git a/addons/promex/service-prometheus.c b/addons/promex/service-prometheus.c
index 4fee8c353..724d24b28 100644
--- a/addons/promex/service-prometheus.c
+++ b/addons/promex/service-prometheus.c
@@ -1617,6 +1617,7 @@ static void promex_appctx_handle_io(struct appctx *appctx)
 	res->flags |= CF_READ_EVENT;
 	sc_shutr(sc);
 	sc_shutw(sc);
+	goto out;
 }
 
 struct applet promex_applet = {