Stephan,<div><br></div><div>I wonder why did I miss that bug. I'll be trying to find a fix.</div><div><br></div><div>Best regards,</div><div><br></div><div>Tamas</div><div><br></div><div><br><br><div class="gmail_quote">
2013/2/26 Stephan Meißl <span dir="ltr"><<a href="mailto:stephan@meissl.name" target="_blank">stephan@meissl.name</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Tamas,<br>
<br>
seems you hit issue #4533. I had an initial look at it but I'm currently<br>
under heavy load and I'm afraid it has to wait a bit longer unless you<br>
want to give it a try?<br>
<br>
Sorry,<br>
Stephan<br>
<div><div class="h5"><br>
<br>
On 02/26/2013 09:31 AM, Tamas Szekeres wrote:<br>
> Hi All,<br>
><br>
> Related to the tests of RFC-90 we found an issue when a layer is<br>
> disabled by IP list, but for the GetCapabilities request the layer is<br>
> re-enabled when the layer is a part of a wms_layer_group. For example<br>
> with the following setting:<br>
><br>
><br>
> LAYER<br>
>    NAME Layer1<br>
>    METADATA<br>
>       "ows_allowed_ip_list" "*"<br>
>       "wms_layer_group" "/Layer1"<br>
>    END<br>
> END<br>
><br>
> LAYER<br>
>    NAME Layer2<br>
>    METADATA<br>
>       "ows_allowed_ip_list" "!*"<br>
>       "wms_layer_group" "/Layer1"<br>
>    END<br>
> END<br>
><br>
> Both Layer1 and Layer2 are displayed in GetCapabilities. The same<br>
> applies when the layer is completely disabled for GetCapabilities by<br>
> enable_request, like:<br>
><br>
><br>
> LAYER<br>
>    NAME Layer1<br>
>    METADATA<br>
>       "wms_enable_request" "GetCapabilities"<br>
>        "wms_layer_group" "/Layer1"<br>
>    END<br>
> END<br>
><br>
> LAYER<br>
>    NAME Layer2<br>
>    METADATA<br>
>       "wms_enable_request" "!GetCapabilities"<br>
>       "wms_layer_group" "/Layer1"<br>
>    END<br>
> END<br>
><br>
> Both Layer1 and Layer2 are displayed.<br>
><br>
> Is this a feature or an issue that should be fixed?<br>
><br>
><br>
> Best regards,<br>
><br>
> Tamas<br>
><br>
><br>
><br>
><br>
><br>
> 2013/2/21 Daniel Morissette <<a href="mailto:dmorissette@mapgears.com">dmorissette@mapgears.com</a><br>
</div></div>> <mailto:<a href="mailto:dmorissette@mapgears.com">dmorissette@mapgears.com</a>>><br>
<div class="im">><br>
>     On 13-02-20 10:05 AM, Tamas Szekeres wrote:<br>
><br>
>         Hi Daniel & All,<br>
><br>
>         I've updated RFC-90 to follow the concept mentioned.<br>
><br>
</div>>         <a href="http://mapserver.org/__development/rfc/ms-rfc-90.html" target="_blank">http://mapserver.org/__development/rfc/ms-rfc-90.html</a><br>
<div class="im">>         <<a href="http://mapserver.org/development/rfc/ms-rfc-90.html" target="_blank">http://mapserver.org/development/rfc/ms-rfc-90.html</a>><br>
><br>
>         Here is a commit containing the proposed changes according to<br>
>         the recent<br>
>         version:<br>
</div>>         <a href="https://github.com/szekerest/__mapserver/commit/__5d7ec08292e4b790d219082f7ea6ce__d83fc5c336" target="_blank">https://github.com/szekerest/__mapserver/commit/__5d7ec08292e4b790d219082f7ea6ce__d83fc5c336</a><br>

<div class="im">>         <<a href="https://github.com/szekerest/mapserver/commit/5d7ec08292e4b790d219082f7ea6ced83fc5c336" target="_blank">https://github.com/szekerest/mapserver/commit/5d7ec08292e4b790d219082f7ea6ced83fc5c336</a>><br>

><br>
><br>
>     Thanks a lot Tamas... the RFC seems to address all my comments now.<br>
><br>
>     BTW, I notice in your commit that you call msOWSLookupMetadata()<br>
>     twice in msOWSIpDisabled():<br>
><br>
><br>
>       ip_list = msOWSLookupMetadata(metadata, namespaces,<br>
>     "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<br>
>     msOWSLookupMetadata() is that all places that pass a "namespaces"<br>
>     arg should include the "O" as the last argument in the string... so<br>
>     your second call with the "O" is redundant unless I'm mistaken since<br>
>     the namespaces in the previous call should contain an "O" already.<br>
><br>
><br>
><br>
>     --<br>
>     Daniel Morissette<br>
>     <a href="http://www.mapgears.com/" target="_blank">http://www.mapgears.com/</a><br>
>     Provider of Professional MapServer Support since 2000<br>
><br>
</div>>     _________________________________________________<br>
>     mapserver-dev mailing list<br>
>     <a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a> <mailto:<a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a>><br>
>     <a href="http://lists.osgeo.org/__mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/__mailman/listinfo/mapserver-dev</a><br>
>     <<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a>><br>
<div class="HOEnZb"><div class="h5">><br>
><br>
><br>
><br>
> _______________________________________________<br>
> mapserver-dev mailing list<br>
> <a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
><br>
<br>
_______________________________________________<br>
mapserver-dev mailing list<br>
<a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
</div></div></blockquote></div><br></div>