2013/4/30 Justin Deoliveira <[email protected]>
> Hi Mauro,
>
> Sorry for the late reply on this. The approach sounds good to me. A couple
> of thoughts and suggestions.
>
> Perhaps it makes sense to have a separate "ActiveDirectory" provider,
> rather than overload the default ldap one with more options. Just a
> suggestion, not sure how much of the existing provider is used in this
> scheme.
>
I thought about that, but I think the new options (particularly the ones
dealing with roles extracting in a bounded context) could be useful for
other kind of LDAP servers too, not only for ActiveDirectory ones.
>
> Regardless of one or more providers, it would be good if we could update
> the docs along with this patch. The relevant page being:
>
>
> http://docs.geoserver.org/stable/en/user/security/auth/providers.html#ldap-authentication
>
Sure, I will do that before making the pull request.
Thanks for the suggestions.
Mauro
--
==
GeoServer training in Milan, 6th & 7th June 2013! Visit
http://geoserver.geo-solutions.it for more information.
==
Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
------------------------------------------------------------------------------
Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
Get 100% visibility into your production application - at no cost.
Code-level diagnostics for performance bottlenecks with <2% overhead
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap1
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
