BUG/MINOR: mux-h2/stats: not all GOAWAY frames are errors
authorWilly Tarreau <w@1wt.eu>
Tue, 1 Dec 2020 09:47:18 +0000 (10:47 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 1 Dec 2020 09:47:18 +0000 (10:47 +0100)
commitf965b2ad136b6487571f9a8fb0b3e58f778c002c
treea218dc789c76e6a86c448d76db929217e1bd14bf
parent5dd36ac8a0984d74e1bb4e3747bc240386e214a6
BUG/MINOR: mux-h2/stats: not all GOAWAY frames are errors

The stats on haproxy.org reported ~12k GOAWAY for ~34k connections, with
only 2 protocol errorss. It turns out that the GOAWAY frame counter added
in commit a8879238c ("MINOR: mux-h2: report detected error on stats")
matches a bit too many situations. First it counts those which are not
sent as well as failed retries, second it counts as errors the cases of
attempts to cleanly close, while it's titled "GOAWAY sent on detected
error". Let's address this by moving the counter up one line and excluding
the clean codes.

This can be backported to 2.3.
src/mux_h2.c