Sounds good, want to make a geoserver branch. And thanks for getting things
started, Kevin also did some research/experiment and may have a
contribution.
Still waiting on PSC votes.
On Wed, Mar 2, 2016 at 10:25 AM Justin Deoliveira <jdeol...@gmail.com>
wrote:
> I took the liberty of starting down the road of this described approach.
> I’ve gotten it to the point where all of the core modules compile (haven’t
> taken on extensions or community modules) against spring 4.2.5 and spring
> security 4.0.4. Haven’t actually run the server or any tests yet.
>
> https://github.com/jdeolive/geoserver/tree/spring4-upgrade
>
> I think the next step would be to merge this with andreas mockrunner
> branch and go ahead and perform the servlet api upgrade. With that done I
> think we would be in a position to start working through all of the unit
> tests.
>
>
> On Wed, Mar 2, 2016 at 8:10 AM Justin Deoliveira <jdeol...@gmail.com>
> wrote:
>
>> The core api changes around ProviderManager and FilterChainProxy I think
>> is something that won’t be too crazy to work around. I think the best bet
>> there will be to have the geoserver counter parts
>> (GeoServerSecurityManager, GeoServerFilterChainProxy) wrap the spring stuff
>> rather than extend it. That way it can be recreated as needed via
>> constructor rather than setters after the fact. In an ideal world nothing
>> around thsoe classes would have to change, but that is probalby a bit too
>> optimistic :)
>>
>> As for the CAS stuff you’re guess is as good as mine. I can confirm
>> Christians findings that the api has changed significantly and from what I
>> can tell without much guidance as to what the transition is from the old
>> api to the new. Could potentially be looking at a re-implementation there
>> but knowing next to nothing about CAS I delegate to Christian on that.
>>
>>
>>
>> On Wed, Mar 2, 2016 at 12:16 AM Christian Mueller <
>> christian.muel...@os-solutions.at> wrote:
>>
>>> Hi all
>>>
>>> The Spring security API changed since 4.x. GeoServer is relying on
>>> getters and setters and this methods are gone. Since 4.x, the instance
>>> variables are set using a constructor. Not easy to change.
>>>
>>> Additionally, the CAS java client also changed the public API, this is
>>> the second problem because the CAS jar is a dependency of Spring Security.
>>>
>>> Will have a look at the problem during the weekend.
>>>
>>> Do we have a branch where I can start investigating ?
>>>
>>> Cheers
>>> Christian
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> On Wed, Mar 2, 2016 at 4:29 AM, Andrea Aime <
>>> andrea.a...@geo-solutions.it> wrote:
>>>
>>>> Hi Jody,
>>>> as said in the meeting, I'm supportive of a delay long enough to switch
>>>> everything to Spring 4 and make it solid (2-3 months)
>>>>
>>>> Cheers
>>>> Andrea
>>>>
>>>>
>>>> On Wed, Mar 2, 2016 at 2:49 AM, Jody Garnett <jody.garn...@gmail.com>
>>>> wrote:
>>>>
>>>>> Okay, I respect both options making this a tough decision. I wish we
>>>>> could hear back from Christian about the security-cas release, but the
>>>>> same
>>>>> spring4 migration needed by both plans.
>>>>>
>>>>> I would like to go ahead with the release delay (do the spring 4
>>>>> upgrade now), avoids an awkward 50% solution that we would need to
>>>>> support.
>>>>>
>>>>>
>>>>> ------------------------------------------------------------------------------
>>>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>>>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>>>>> Monitor end-to-end web transactions and take corrective actions now
>>>>> Troubleshoot faster and improve end-user experience. Signup Now!
>>>>> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>>>>> _______________________________________________
>>>>> Geoserver-devel mailing list
>>>>> Geoserver-devel@lists.sourceforge.net
>>>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> ==
>>>> GeoServer Professional Services from the experts! Visit
>>>> http://goo.gl/it488V for more information.
>>>> ==
>>>>
>>>> Ing. Andrea Aime
>>>> @geowolf
>>>> Technical Lead
>>>>
>>>> GeoSolutions S.A.S.
>>>> Via di Montramito 3/A
>>>> 55054 Massarosa (LU)
>>>> phone: +39 0584 962313
>>>> fax: +39 0584 1660272
>>>> mob: +39 339 8844549
>>>>
>>>> http://www.geo-solutions.it
>>>> http://twitter.com/geosolutions_it
>>>>
>>>> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>>>>
>>>> Le informazioni contenute in questo messaggio di posta elettronica e/o
>>>> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
>>>> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
>>>> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
>>>> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
>>>> darcene notizia via e-mail e di procedere alla distruzione del messaggio
>>>> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
>>>> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
>>>> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
>>>> principi dettati dal D.Lgs. 196/2003.
>>>>
>>>>
>>>>
>>>> The information in this message and/or attachments, is intended solely
>>>> for the attention and use of the named addressee(s) and may be confidential
>>>> or proprietary in nature or covered by the provisions of privacy act
>>>> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
>>>> Code).Any use not in accord with its purpose, any disclosure, reproduction,
>>>> copying, distribution, or either dissemination, either whole or partial, is
>>>> strictly forbidden except previous formal approval of the named
>>>> addressee(s). If you are not the intended recipient, please contact
>>>> immediately the sender by telephone, fax or e-mail and delete the
>>>> information in this message that has been received in error. The sender
>>>> does not give any warranty or accept liability as the content, accuracy or
>>>> completeness of sent messages and accepts no responsibility for changes
>>>> made after they were sent or for other risks which arise as a result of
>>>> e-mail transmission, viruses, etc.
>>>>
>>>> -------------------------------------------------------
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>>>> Monitor end-to-end web transactions and take corrective actions now
>>>> Troubleshoot faster and improve end-user experience. Signup Now!
>>>> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>>>> _______________________________________________
>>>> Geoserver-devel mailing list
>>>> Geoserver-devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>>
>>>>
>>>
>>>
>>> --
>>> DI Christian Mueller MSc (GIS), MSc (IT-Security)
>>> OSS Open Source Solutions GmbH
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>>> Monitor end-to-end web transactions and take corrective actions now
>>> Troubleshoot faster and improve end-user experience. Signup Now!
>>> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
>>> _______________________________________________
>>> Geoserver-devel mailing list
>>> Geoserver-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>>>
>>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
> _______________________________________________
> Geoserver-devel mailing list
> Geoserver-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
--
--
Jody Garnett
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
