On 25/01/22 17:38, Alex Rousskov wrote:
On 1/24/22 7:24 PM, Praveen Ponakanti wrote:
The proposed solution seems to be to change the following lines and
recompile. If this is still the recommended fix, can it be upstreamed in
an upcoming release?

https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6042
https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6136

The above change of squid.conf defaults will probably work for you, but
it will also break some deployments affected by this bug because the
current code implicitly relies on those defaults triggering lookups.
Thus, it is probably not the right solution for upstreaming. The PR
mentioned below discusses specifics (see item #1 in the PR description).


Alex: FWIW; the above breakage is the same reason for my remaining objection to PR #912.


Amos
_______________________________________________
squid-users mailing list
squid-users@lists.squid-cache.org
http://lists.squid-cache.org/listinfo/squid-users

Reply via email to