]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: quic: Do not ack when probing
authorFrédéric Lécaille <flecaille@haproxy.com>
Tue, 30 Aug 2022 14:24:54 +0000 (16:24 +0200)
committerWilly Tarreau <w@1wt.eu>
Fri, 2 Sep 2022 06:47:38 +0000 (08:47 +0200)
commitb866c69f4f148bfb2e02fcf44970f5774afb097e
tree64fffa93c26ad4bebbc077b63652c3f0eadc483b
parentecb40b2c388dffcc9998a92df536ec7419ed8fed
BUG/MINOR: quic: Do not ack when probing

<force_ack> boolean variable passed to qc_do_build_pkt() which builds a clear
packet is there to force this function to build an ACK frame regardless of
others conditions. This is used during handshake, when we acknowledge every
handshake packets received.

This variable was already taken into an account by the local variable <must_ack>
which is there at least to ignore any other conditions than this one: "are
we building a probing packet?". Indeed we do not want to add ACK frames when
we probe the peers. This is to have more chances to embed the new duplicated frames
into another packets without splitting them. So, the test on <force_ack> boolean
value is useless, silly and brakes the rule which consists in not acknowledging
when probing.

Must be backported to 2.6.
src/xprt_quic.c