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 edee3aa..3e1b90b 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -4506,7 +4506,8 @@
}
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);
}