<div dir="ltr">Confirm patch works, in both formats. <div>Thanks !</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 1, 2014 at 8:19 AM, Ali Jawad <span dir="ltr"><<a href="mailto:alijawad1@gmail.com" target="_blank">alijawad1@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Hi Evan <br>
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 !</p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On May 1, 2014 6:38 AM, "Evan Hunt" <<a href="mailto:each@isc.org" target="_blank">each@isc.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


On Wed, Apr 30, 2014 at 11:33:06PM +0200, Ali Jawad wrote:<br>
> Any hints ?<br>
<br>
So, yeah, that's embarrassing.  It never crossed my mind to test geoip<br>
ACL elements by referencing them indirectly in named ACLs, as you did;<br>
I only referenced them directly. Apparently none of the folks who've<br>
been using the code in production ever tried that either. Kudos for<br>
your QA skills.  :)<br>
<br>
Thanks to you, I am now aware of the fact that, while the following<br>
configuration does work:<br>
<br>
        match-clients { geoip country US; };<br>
<br>
...this one doesn't:<br>
<br>
        acl geoipUS { geoip country US; }<br>
        ...<br>
        match-clients { geoipUS; };<br>
<br>
The problem is that when the "geoipUS" ACL is merged into match-clients<br>
for the view, the geoip information doesn't get copied correctly.<br>
<br>
The attached patch should fix it.<br>
<br>
--<br>
Evan Hunt -- <a href="mailto:each@isc.org" target="_blank">each@isc.org</a><br>
Internet Systems Consortium, Inc.<br>
</blockquote></div>
</div></div></blockquote></div><br></div>