From a4dcd3e54b12d9d313801938cfc30141c0bdfbef Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Tue, 23 Jan 2024 08:22:53 +0100 Subject: [PATCH] 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. --- src/stats.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/stats.c b/src/stats.c index 6ef8fec5a..7c1f34961 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) {