From: Christopher Faulet <cfaulet@haproxy.com>
Date: Tue, 28 Mar 2023 10:02:03 +0000 (+0200)
Subject: BUG/MINOR: stats: Don't replace sc_shutr() by SE_FL_EOS flag yet
X-Git-Tag: v2.8-dev7~147
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=41a454da0a47e0d37e37f2d9a13712b7736e9cc5;p=thirdparty%2Fhaproxy.git

BUG/MINOR: stats: Don't replace sc_shutr() by SE_FL_EOS flag yet

In commit c2c043ed4 ("BUG/MEDIUM: stats: Consume the request except when
parsing the POST payload"), a change about applet was pushed too early. The
applet must still call cf_shutr() when the response is fully sent. It is
planned to rely on SE_FL_EOS flag, just like connections. But it is not
possible for now.

However, at first glance, this bug has no visible effect.

It is 2.8-specific. No backport needed.
---

diff --git a/src/stats.c b/src/stats.c
index edee3aaf5e..3e1b90b1f0 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -4506,7 +4506,8 @@ static void http_stats_io_handler(struct appctx *appctx)
 	}
 
 	if (appctx->st0 == STAT_HTTP_END) {
-		se_fl_set(appctx->sedesc, SE_FL_EOS);
+		if (!(res->flags & CF_SHUTR))
+			sc_shutr(sc);
 		applet_will_consume(appctx);
 	}