Merge lp://staging/~sdeziel/squid-deb-proxy/remove-unused-acls into lp://staging/squid-deb-proxy
Proposed by
Simon Déziel
Status: | Merged |
---|---|
Merged at revision: | 70 |
Proposed branch: | lp://staging/~sdeziel/squid-deb-proxy/remove-unused-acls |
Merge into: | lp://staging/squid-deb-proxy |
Diff against target: |
14 lines (+0/-4) 1 file modified
squid-deb-proxy.conf (+0/-4) |
To merge this branch: | bzr merge lp://staging/~sdeziel/squid-deb-proxy/remove-unused-acls |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Vogt (community) | Needs Information | ||
Review via email:
|
Description of the change
The provided squid-deb-
Since the config parser complains on startup and during log rotation, the useless ACL should be removed to avoid spamming the admin with mesages like these :
/etc/cron.
2012/04/20 06:25:14| WARNING: (B) '::/0' is a subnetwork of (A) '::/0'
2012/04/20 06:25:14| WARNING: because of this '::/0' is ignored to keep splay tree searching predictable
2012/04/20 06:25:14| WARNING: You should probably remove '::/0' from the ACL named 'all'
Thanks
To post a comment you must log in.
Thanks for your branch! From reading the linked mail it appears this is done in squid3, but currently squid-deb-proxy supports squid2 as well. Would removing those ACLs cause issues with squid2?