]> git.ipfire.org Git - thirdparty/squid.git/commit
Simplified quick_abort_pct code and improved its docs (#1921)
authorShailesh Vashishth <shavashishth@gmail.com>
Sat, 23 Nov 2024 13:41:03 +0000 (13:41 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Sat, 23 Nov 2024 17:27:24 +0000 (17:27 +0000)
commit79600062a8e97e095a96b6c1b7d8bc2b8e17dcce
tree8df75ad1a3386f12f668e33d34158b06dd92e622
parent4531b5c95dda51a49d7d20a155af265b6ede3126
Simplified quick_abort_pct code and improved its docs (#1921)

Instead of ignoring quick_abort_pct settings that would, together with
other conditions, abort a pending download of a 99-byte or smaller
response, Squid now honors quick_abort_pct for all response sizes. Most
Squids are not going to be affected by this change because default
quick_abort_min settings (16KB) prevent aborts of 99-byte responses even
before quick_abort_pct is checked.

Due to conversion from integer to floating point math, this change may
affect responses larger than 99 bytes as well, but these effects ought
to be limited to cases where the decision is based on a tiny difference
(e.g., receiving 1% more bytes would have triggered full download). In
most such cases, the decision could probably go either way due to
response header size fluctuations anyway.

Also updated quick_abort_pct documentation, primarily to clarify a
misleading statement: Squid did not and does not treat 16KB or smaller
responses specially in this context. The original statement was probably
based on quick_abort_min _default_ setting of 16KB, but statement
phrasing and placement hid that connection.
CONTRIBUTORS
doc/release-notes/release-7.sgml.in
src/cf.data.pre
src/store_client.cc