[BUG] http_process_res_common() must not skip the forward analyser
By default this function returned 0 indicating an end of analysis.
This was not a problem as long as it was the last analyser in the
chain but becomes quite a big one now since it skips the forwarder
with auto_close enabled, causing some data to pass under the nose
of the last one undetected.
diff --git a/src/proto_http.c b/src/proto_http.c
index 03e730a..c68d49d 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -4267,10 +4267,9 @@
}
/* This function performs all the processing enabled for the current response.
- * It normally returns zero, but may return 1 if it absolutely needs to be
- * called again after other functions. It relies on buffers flags, and updates
- * t->rep->analysers. It might make sense to explode it into several other
- * functions. It works like process_request (see indications above).
+ * It normally returns 1 unless it wants to break. It relies on buffers flags,
+ * and updates t->rep->analysers. It might make sense to explode it into several
+ * other functions. It works like process_request (see indications above).
*/
int http_process_res_common(struct session *t, struct buffer *rep, int an_bit, struct proxy *px)
{
@@ -4644,9 +4643,9 @@
* otherwise we would not let the client side wake up.
*/
- return 0;
+ return 1;
}
- return 0;
+ return 1;
}
/* This function is an analyser which forwards response body (including chunk