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

BUG/MEDIUM: check: Requeue healthchecks on I/O events to handle check timeout

Browse Source 
When a healthchecks is processed, once the first wakeup passed to start the
check, and as long as the expiration timer is not reached, only I/O events
are able to wake it up. It is an issue when there is a check timeout
defined.  Especially if the connect timeout is high and the check timeout is
low. In that case, the healthcheck's task is never requeue to handle any
timeout update. When the connection is established, the check timeout is set
to replace the connect timeout. It is thus possible to report a success
while a timeout should be reported.

So, now, when an I/O event is handled, the healthcheck is requeue, except if
an success or an abort is reported.

Thanks to Thierry Fournier for report and the reproducer.

This patch must be backported to all stable versions.
This commit is contained in:
Christopher Faulet 2025-06-03 14:50:38 +02:00
parent 913b2d6c83
commit 7c788f0984
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/check.c
View File

@ -1088,6 +1088,11 @@ int wake_srv_chk(struct stconn *sc)
ret = -1;
task_wakeup(check->task, TASK_WOKEN_IO);
}
else {
/* Check in progress. Queue it to eventually handle timeout
* update */
task_queue(check->task);
}
if (check->server)
HA_SPIN_UNLOCK(SERVER_LOCK, &check->server->lock);