BUG/MEDIUM: stconn: Don't forward shut for SC in connecting state
authorChristopher Faulet <cfaulet@haproxy.com>
Wed, 30 Oct 2024 15:41:39 +0000 (16:41 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 13 Nov 2024 09:53:27 +0000 (10:53 +0100)
commit72e529829b8eedebc1891970cf8ac64eaf26d006
treef9617ecdba8a57b014801fb8abfeb8e7a9fabb5b
parent113745e6f0c0ef8fe89e89fdfdcc6ed994889d4a
BUG/MEDIUM: stconn: Don't forward shut for SC in connecting state

In connecting state, shutdown must not be forwarded or scheduled because
otherwise this will prevent any connection retries. Indeed, if a EOS is
reported by the mux during the connection establishment, this should be
handled by the stream to eventually retries. If the write side is closed
first, this will not be possible because the stconn will be switched in DIS
state. If the shut is scheduled because pending data are blocked, the same
may happen, depending on the abort-on-close option.

This patch should be slowly be backported as far as 2.4. But an observation
period is mandatory. On 2.4, the patch must be adapted to use the
stream-interface API.
src/stconn.c