From 58a81aeb91bc0c3b07f340852c09af3b5692899a Mon Sep 17 00:00:00 2001
From: William Lallemand <wlallemand@haproxy.org>
Date: Thu, 17 Mar 2022 15:14:15 +0100
Subject: [PATCH] BUG/MINOR: httpclient: CF_SHUTW_NOW should be tested with
 channel_is_empty()

CF_SHUTW_NOW shouldn't be a condition alone to exit the io handler, it
must be tested with the emptiness of the response channel.

Must be backported to 2.5.
---
 src/http_client.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/http_client.c b/src/http_client.c
index 39d7aa2ac..fec35c1b6 100644
--- a/src/http_client.c
+++ b/src/http_client.c
@@ -928,7 +928,8 @@ more:
 	 * isn't any data to handle and a shutdown is detected, let's stop
 	 * everything */
 	if ((req->flags & (CF_SHUTR|CF_SHUTR_NOW)) ||
-	    (res->flags & (CF_SHUTW|CF_SHUTW_NOW))) {
+	    (res->flags & CF_SHUTW) ||
+	    ((res->flags & CF_SHUTW_NOW) && channel_is_empty(res))) {
 		goto end;
 	}
 	return;