1berry/haproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MINOR: mux-h2: do not emit GOAWAY on idle ping expiration

Browse Source 
If idle-ping is activated and h2c task is expired due to missing PING
ACK, consider that the peer is away and the connection can be closed
immediately. GOAWAY emission is thus skipped.

A new test is necessary in h2c_update_timeout() when PING ACK is
currently expected, but the next timer expiration selected is not
idle-ping. This may happen if http-keep-alive/http-request timers are
selected first. In this case, H2_CF_IDL_PING_SENT flag is resetted. This
is necessary to not prevent GOAWAY emission on expiration.
This commit is contained in:
Amaury Denoyelle 2025-04-17 11:14:10 +02:00
parent 52246249ab
commit cc5a7a760f
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/mux_h2.c
View File

@ -914,10 +914,18 @@ static void h2c_update_timeout(struct h2c *h2c)
ping = tick_add_ifset(now_ms, conn_idle_ping(h2c->conn));
exp = tick_first(exp, ping);
/* If PING timer selected, set flag to trigger its emission rather than conn deletion on next timeout. */
if (tick_isset(exp) && exp == ping && ping != dft &&
!(h2c->flags & H2_CF_IDL_PING_SENT)) {
h2c->flags |= H2_CF_IDL_PING;
if (tick_isset(exp)) {
if (exp == ping && ping != dft) {
/* PING timer selected, set flag to prevent conn deletion on next timeout. */
if (!(h2c->flags & H2_CF_IDL_PING_SENT))
h2c->flags |= H2_CF_IDL_PING;
}
else if (h2c->flags & H2_CF_IDL_PING_SENT) {
/* timer other than ping selected, remove ping flag to allow GOAWAY on expiration. */
h2c->flags &= ~H2_CF_IDL_PING_SENT;
ABORT_NOW();
}
}
}
@ -5220,6 +5228,12 @@ struct task *h2_timeout_task(struct task *t, void *context, unsigned int state)
HA_SPIN_UNLOCK(IDLE_CONNS_LOCK, &idle_conns[tid].idle_conns_lock);
if (h2c->flags & H2_CF_IDL_PING_SENT) {
TRACE_STATE("expired on idle ping", H2_EV_H2C_WAKE, h2c->conn);
/* Do not emit GOAWAY on idle ping expiration. */
goto do_leave;
}
/* Try to gracefully close idle connections by sending a GOAWAY first,
* and then waiting for the fin timeout.
*/