tree cbdab06c121544f7ead96db5f9812545ec501eca
parent 1d66a771d48529ac9312d950e86dbc77d4c6a617
author Willy Tarreau <w@1wt.eu> 1602605355 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1603089906 +0200

BUG/MINOR: mux-h2: do not stop outgoing connections on stopping

There are reports of a few "SC" in logs during reloads when H2 is used
on the backend side. Christopher analysed this as being caused by the
proxy disabled test in h2_process(). As the comment says, this was done
for frontends only, and must absolutely not send a GOAWAY to the backend,
as all it will result in is to make newly queued streams fail.

The fix consists in simply testing the connection side before deciding
to send the GOAWAY.

This may be backported as far as 2.0, though for whatever reason it seems
to manifest itself only since 2.2 (probably due to changes in the outgoing
connection setup sequence).

(cherry picked from commit b1e600c9c5e4a20a6de1f3b9593f20dc50f01315)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 403168c3787cc7098fe31640089bd508d3448115)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 70b0527f294939a21bf9819b941bf6a05a5db9c9)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>