Right, the virtual servies manager is a wrapper. I believe we put it in the
app context to make it easy to disable on 2.0.x where we did not want it to
engage by default (although the client that funded the work did need it on
2.0.x). So I am all for being explicit about creating it in
SecureCatalogImpl.
On Wed, Feb 16, 2011 at 11:34 AM, Andrea Aime
<[email protected]>wrote:
>
> On Wed, Feb 16, 2011 at 7:04 PM, Justin Deoliveira
> <[email protected]>wrote:
>
>> This is most likely due to the virtual services stuff which installs its
>> own DataAccessManager. And as you can see the lookup is not prepared to
>> handle multiple access managers. If i remember correctly when i made that
>> change I figured this might come up.
>>
>> Not sure how to best handle this. My first thought would be to do a look
>> up for all of them in the application context and wrap them up in a
>> composite manager. But would like to hear Andrea weigh in.
>>
>
> Yes, the plugin system was created so that there can be only one security
> manager around.
> Which makes sense, you don't want to have a russian roulette of what the
> security subsystem will be in case
> there is more than one, depending on the JVM chosen, the container and the
> eventual alignment of planets in
> the solar system ;-)
>
> Now, the virtual services thing as far as I can see is there to wrap and
> alter the security settings, it's not
> really the security manager, it's a just wrapper around it.
>
> So imho it should not be part of the app context, but be created around the
> chosen security manager
> by the secure catalog impl instead.
>
> Btw, I did not touch that part since I did not know exactly what it was
> doing, but the wrapper should
> be updated to be a ResourceAccessManager instead
>
> Cheers
> Andrea
>
> --
> -------------------------------------------------------
> Ing. Andrea Aime
> GeoSolutions S.A.S.
> Tech lead
>
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
>
> phone: +39 0584 962313
> fax: +39 0584 962313
> mob: +39 333 8128928
>
> http://www.geo-solutions.it
> http://geo-solutions.blogspot.com/
> http://www.youtube.com/user/GeoSolutionsIT
> http://www.linkedin.com/in/andreaaime
> http://twitter.com/geowolf
>
> -------------------------------------------------------
>
--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
