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 6ef8fec5ad45b07e483d617e128f87b0737e3250..7c1f349618283badfc4154275670062d3a2b9b25 100644 (file)
--- 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) {