[squid-dev] [PATCH] Remove cache_peer_domain

[Kinkie]

+1.

On Sun, Feb 1, 2015 at 7:22 PM, Amos Jeffries <squid3 at treenet.co.nz> wrote:
> The cache_peer_domain directive functionality is also provided through
> cache_peer_access.
>
> While this check appears at face value to be simpler than ACLs, the
> reality is that:
> * the difference is simply the time it takes to initialize and destruct
> an on-stack Checklist,
> * processing the checks may take longer than ACLs (linked-list of string
> comparisons vs single tree lookup)
> * ACLs are the common case due to their extra flexibility, and
> * extra work is being done per-transaction just to check which of the
> two features is in use.
>
> Overall I just dont believe it to be worth keeping. By removing we gain
> less code and configuration directives to work around in the long term.
>
> Amos
>
> _______________________________________________
> squid-dev mailing list
> squid-dev at lists.squid-cache.org
> http://lists.squid-cache.org/listinfo/squid-dev
>



-- 
    Francesco