Hi Evan
You guys are doing a great job. I did actually try before without acl
directly in view. The error was at the time No ACL geoip. Will retry with
patch. Thanks !
On May 1, 2014 6:38 AM, "Evan Hunt" <e...@isc.org> wrote:
> On Wed, Apr 30, 2014 at 11:33:06PM +0200, Ali Jawad wrote:
> > Any hints ?
>
> So, yeah, that's embarrassing. It never crossed my mind to test geoip
> ACL elements by referencing them indirectly in named ACLs, as you did;
> I only referenced them directly. Apparently none of the folks who've
> been using the code in production ever tried that either. Kudos for
> your QA skills. :)
>
> Thanks to you, I am now aware of the fact that, while the following
> configuration does work:
>
> match-clients { geoip country US; };
>
> ...this one doesn't:
>
> acl geoipUS { geoip country US; }
> ...
> match-clients { geoipUS; };
>
> The problem is that when the "geoipUS" ACL is merged into match-clients
> for the view, the geoip information doesn't get copied correctly.
>
> The attached patch should fix it.
>
> --
> Evan Hunt -- e...@isc.org
> Internet Systems Consortium, Inc.
>
_______________________________________________
Please visit https://lists.isc.org/mailman/listinfo/bind-users to unsubscribe
from this list
bind-users mailing list
bind-users@lists.isc.org
https://lists.isc.org/mailman/listinfo/bind-users
