<div dir="ltr">Thanks Alex. I will try out the change with the config defaults in cd.data.pre in a test env for now and wait for the PR with the fix to be upstreamed.<div>Praveen</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 24, 2022 at 8:38 PM Alex Rousskov <<a href="mailto:rousskov@measurement-factory.com">rousskov@measurement-factory.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/24/22 7:24 PM, Praveen Ponakanti wrote:<br>
> Hi,<br>
> <br>
> I am running squid version 4.17 and have not been able to disable the<br>
> reverse DNS lookups it does on each client's IP address. Found the<br>
> thread below that discusses this; I have attempted adding the following<br>
> config knobs, but it still does not disable the reverse lookups. I do<br>
> not have any logformat containing ">A" or "<A". <br>
> <br>
> store_id_extras "%>a %un %>rm myip=%la myport=%lp"<br>
> url_rewrite_extras "%>a %un %>rm myip=%la myport=%lp"<br>
> <br>
> <a href="http://lists.squid-cache.org/pipermail/squid-users/2016-February/009109.html" rel="noreferrer" target="_blank">http://lists.squid-cache.org/pipermail/squid-users/2016-February/009109.html</a><br>
> <br>
> The proposed solution seems to be to change the following lines and<br>
> recompile. If this is still the recommended fix, can it be upstreamed in<br>
> an upcoming release?<br>
> <br>
> <a href="https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6042" rel="noreferrer" target="_blank">https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6042</a><br>
> <a href="https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6136" rel="noreferrer" target="_blank">https://github.com/squid-cache/squid/blob/master/src/cf.data.pre#L6136</a><br>
<br>
The above change of squid.conf defaults will probably work for you, but<br>
it will also break some deployments affected by this bug because the<br>
current code implicitly relies on those defaults triggering lookups.<br>
Thus, it is probably not the right solution for upstreaming. The PR<br>
mentioned below discusses specifics (see item #1 in the PR description).<br>
<br>
<br>
The solution proposed for upstreaming is at<br>
<a href="https://github.com/squid-cache/squid/pull/912" rel="noreferrer" target="_blank">https://github.com/squid-cache/squid/pull/912</a><br>
<br>
It does not require changing squid.conf defaults and has no (known to<br>
me) other serious flaws.<br>
<br>
<br>
HTH,<br>
<br>
Alex.<br>
</blockquote></div>