From: Christopher Faulet <cfaulet@haproxy.com>
Date: Thu, 1 Feb 2018 07:45:45 +0000 (+0100)
Subject: BUG/MEDIUM: spoe: Allow producer to read and to forward shutdown on request side
X-Git-Tag: v1.9-dev1~463
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=9cdca976d325cfc69a085b7d9e53ee1789a18a5b;p=thirdparty%2Fhaproxy.git

BUG/MEDIUM: spoe: Allow producer to read and to forward shutdown on request side

This is mandatory to correctly set right timeout on the stream. Else the client
timeout is never set. So only SPOE processing timeout will be evaluated. If it
is not defined (ie infinity), the stream can be blocked for a while, waiting the
SPOA reply. Of course, this is not a good idea to let the SPOE processing
timeout undefined, but it can happen.

This patch must be backported in 1.8.
---

diff --git a/src/flt_spoe.c b/src/flt_spoe.c
index 5b6e25370b..15317d1b87 100644
--- a/src/flt_spoe.c
+++ b/src/flt_spoe.c
@@ -3088,7 +3088,7 @@ spoe_chn_pre_analyze(struct stream *s, struct filter *filter,
 	}
 
   out:
-	if (!ret) {
+	if (!ret && (chn->flags & CF_ISRESP)) {
                 channel_dont_read(chn);
                 channel_dont_close(chn);
 	}