<br><br><div class="gmail_quote">2013/2/21 Daniel Morissette <span dir="ltr"><<a href="mailto:dmorissette@mapgears.com" target="_blank">dmorissette@mapgears.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br></div>
<br>
BTW, I notice in your commit that you call msOWSLookupMetadata() twice in msOWSIpDisabled():<br>
<br>
<br>
  ip_list = msOWSLookupMetadata(metadata, namespaces, "allowed_ip_list");<br>
<br>
  if (!ip_list)<br>
    ip_list = msOWSLookupMetadata(metadata, "O", "allowed_ip_list");<br>
<br>
I'm not sure why you do this. FYI the idea behind msOWSLookupMetadata() is that all places that pass a "namespaces" arg should include the "O" as the last argument in the string... so your second call with the "O" is redundant unless I'm mistaken since the namespaces in the previous call should contain an "O" already.<div class="HOEnZb">
<div class="h5"><br></div></div></blockquote><div><br></div><div>Hi Daniel,</div><div><br></div><div>Since msOWSRequestIsEnabled and msOWSRequestLayersEnabled is called from various places (mapogcsos.c, mapwcs.c, mapwfs.c, mapwms.c) the passed argument (namespaces) doesn't necessarily contain the "O" setting. This is similar how msOWSRequestIsEnabled and msOWSRequestLayersEnabled already contains this additional metadata lookup.</div>
<div><br></div><div>Best regards,</div><div><br></div><div>Tamas</div><div><br></div></div>