Scott McCrory wrote:
> Ben Alex wrote:
>> Could other developers please finalize their 1.0.2-related tasks (see
>> http://opensource.atlassian.com/projects/spring/secure/BrowseProject.jspa).
>>   
> 
> Ben, I'd like to get the Siteminder improvements noted in SEC-319 in 
> with the 1.0.2 release if permissible.  The fix version was set for 
> 1.1.0, but that was before the 1.0.2 and 1.0.3 releases were added to 
> Jira.  I already have changes committed in SVN that work, but I've made 
> improvements to both the code and documentation since then that I'd like 
> to commit and make complete.  Anyone opposed?

Hi Scott

As we're now post-1.0.0, it's important that we follow the APR
versioning guidelines which state that patch releases (ie 1.0.x) should
be binary and source compatible with previous releases in that series.
In other words, people should be able to simply "drop in" the new JAR
and it work.

Just looking at the revision history for
SiteminderAuthenticationProvider and its corresponding tests, they seem
to be new classes added 27 July 2006. As such, I imagine that users
employing the 1.0.1 SiteMinder integration will need to change their
configuration to use these new classes, and in doing so not benefit from
a "drop in" replacement.

I don't think SiteMinder usage with Acegi Security is extremely
widespread, so we could relax the rules a little if there is good reason
to include the SEC-319 changes in 1.0.2. The conservative choice would
be to defer until 1.1.0, though (assuming I haven't misunderstood the
backward compatibility issue - if the existing integration continues to
work in 1.0.2, I have no problem at all with the refactoring being
included so people have the choice of using it if they wish).

Cheers
Ben

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Home: http://acegisecurity.org
Acegisecurity-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acegisecurity-developer

Reply via email to