From: Christopher Faulet <cfaulet@haproxy.com>
Date: Tue, 23 Jan 2024 07:22:53 +0000 (+0100)
Subject: MEDIUM: stats: Don't interrupt processing on partial post
X-Git-Tag: v3.0-dev3~52
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a4dcd3e54b12d9d313801938cfc30141c0bdfbef;p=thirdparty%2Fhaproxy.git

MEDIUM: stats: Don't interrupt processing on partial post

We no longer test the opposite stream-connector to detect aborted partial
post. Applets must not try to access to info ouside their scope. This make
the code more sensitive to changes and it is a common source of bug.

Tests on the sedesc flags at the begining of the I/O handler should be
enough.
---

diff --git a/src/stats.c b/src/stats.c
index 6ef8fec5ad..7c1f349618 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -4524,8 +4524,6 @@ static void http_stats_io_handler(struct appctx *appctx)
 	if (appctx->st0 == STAT_HTTP_POST) {
 		if (stats_process_http_post(sc))
 			appctx->st0 = STAT_HTTP_LAST;
-		else if (s->scf->flags & (SC_FL_EOS|SC_FL_ABRT_DONE))
-			appctx->st0 = STAT_HTTP_DONE;
 	}
 
 	if (appctx->st0 == STAT_HTTP_LAST) {