mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-08-06 23:27:04 +02:00
Code Issues Projects Releases Wiki Activity

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

Browse Source 
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.
This commit is contained in:
Christopher Faulet 2023-03-28 12:02:03 +02:00
parent 4c7588dd22
commit 41a454da0a
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/stats.c
View File

@ -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);
}