<div dir="ltr"><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 7 Jul 2020 at 18:53, 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 7/7/20 10:52 AM, Matthew Macdonald-Wallace wrote:<br>
<br>
> We're re-configuring a squid proxy solution for a client and as part of<br>
> it we made the assumption that squid would fail if we asked it to read a<br>
> whitelist that wasn't present.<br>
> <br>
> We've now discovered that Squid fails to read the file, throws an error<br>
> in the log ( Error: Cannot open file /etc/squid/whitelist.txt for<br>
> reading ), and then starts up anyway<br>
<br>
Yes, this kind of error ignorance is an old known Squid problem. Some<br>
developers have thought that it is better to start Squid "if at all<br>
possible" than to fail on (in their view "minor") error. New features<br>
are usually more "conservative", but even now that "conservative"<br>
approach does not always win.<br>
<br>
IMO, quality pull requests making missing files a fatal configuration<br>
error should be welcomed. They may not be backported to stable versions,<br>
of course. The solution would probably revolve around throwing an<br>
exception in ConfigParser::strtokFile(). Making missing file treatment<br>
configurable, especially on a per-file basis should be welcomed as well,<br>
probably by extending the new parameters syntax mentioned below.<br>
<br></blockquote><div><br></div><div>Thanks, it did seem a bit odd as a default behaviour, good to know something like this would be welcomed (by you at least!).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Meanwhile, try using the newer parameters() syntax instead of abusing<br>
double quotes. It should work the way you expect. Here is the<br>
corresponding quote from squid.conf.documented:<br>
<br>
> Squid supports reading configuration option parameters from external<br>
> files using the syntax:<br>
>     parameters("/path/filename")<br>
> For example:<br>
>     acl whitelist dstdomain parameters("/etc/squid/whitelist.txt")<br><br></blockquote><div><br></div><div>I'll check the version that we're running and see if I can do this.  I suspect that due to "enterprise requirements" our version won't be the latest, but hopefully it will support this.</div><div><br></div><div>Thanks again,</div><div><br></div><div>Matt </div></div></div>

<br>
<div><br>--</div><div>Mockingbird Consulting<br></div><div>Connecting you with your environment</div><div><br></div><div dir="ltr">w: <a href="http://www.mockingbirdconsulting.co.uk/" target="_blank">www.mockingbirdconsulting.<wbr>co.uk</a><br>e: <a href="mailto:matt@mockingbirdconsulting.co.uk" target="_blank">info@mockingbirdconsulting.<wbr>co.uk</a><br>t: +44 (0) 1600 717142</div><div dir="ltr"><br></div><div dir="ltr">Bridges Centre,</div><div dir="ltr">Drybridge House,</div><div dir="ltr">Monmouth,</div><div dir="ltr">NP25 5AS<br><br>Registered in England and Wales, Company Number 10488438</div>