]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MINOR: checks: do not call conn_xprt_stop_send() anymore
authorWilly Tarreau <w@1wt.eu>
Fri, 21 Feb 2020 09:13:03 +0000 (10:13 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 21 Feb 2020 10:21:12 +0000 (11:21 +0100)
commit7c9d0e1b20ec5e70089be1d526a75f7092b54a6e
tree0318034517fe6880d8de0215bda340bd0d5e66c5
parentd57e34978db7482ebdb0557f2908a033c40394c2
MINOR: checks: do not call conn_xprt_stop_send() anymore

While trying to address issue #253, Commit 5909380c ("BUG/MINOR: checks:
stop polling for write when we have nothing left to send") made sure that
we stop polling for writes when the buffer is empty. This was actually
more a workaround than a bug fix because by doing so we may be stopping
polling for an intermediary transport layer without acting on the check
itself in case there's SSL or send-proxy in the chain for example, thus
the approach is wrong. In practice due to the small size of check
requests, this will not have any impact. At best, we ought to unsubscribe
for sending, but that's already the case when we arrive in this function.
But given that the root cause of the issue was addressed later in commits
cc705a6bc5940392 and ccf3f6d1, we can now safely revert this change.

It was confirmed on the faulty config that this change doesn't have any
effect anymore on the test.
src/checks.c