On Thu, Jun 11, 2009 at 2:43 PM, Sai Pullabhotla<sai.pullabho...@jmethods.com> wrote: > Currently the UserManager interface has the authenticate method defined as > follows: > > User authenticate(Authentication authentication) > throws AuthenticationFailedException; > > I'm wondering if it would be of any benefit to change it to: > > User authenticate(Authentication authentication, FtpSession session) > throws AuthenticationFailedException;
I think this is an excellent change (I got a bunch of changes to UserManager that I would like to do as well). However, since we now got a freezed API, we can not change this in 1.x. How about adding a JIRA issue for this change for a future 2.0 and let's start think whatever else we would like to change in 2.0? /niklas