Thanks, Niklas. I thought there was something like that, but overlooked. Sai Pullabhotla www.jMethods.com
On Mon, Jun 29, 2009 at 7:25 AM, Niklas Gustavsson (JIRA) <j...@apache.org>wrote: > > [ > https://issues.apache.org/jira/browse/FTPSERVER-315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12725150#action_12725150] > > Niklas Gustavsson commented on FTPSERVER-315: > --------------------------------------------- > > The client certificate chain is already available to the UserManager in the > UserMetadata class (calling > authentication.getUserMetadata().getCertificateChain()). > > > Pass FtpSession information to the UserManager.authenticate method > > ------------------------------------------------------------------ > > > > Key: FTPSERVER-315 > > URL: https://issues.apache.org/jira/browse/FTPSERVER-315 > > Project: FtpServer > > Issue Type: Improvement > > Components: Core, Ftplets > > Reporter: Sai Pullabhotla > > Priority: Minor > > Fix For: 2.0.0 > > > > > > 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; > > The reason(s) behind this - > > I want to log a message when the login fails. The login could fail to due > to a number of reasons - such as Account is disabled, password has expired > and so on. Since I do not have the session information available from this > interface, I'm not able to log all the information that I normally do - such > as the session ID, remote address and so on. I know I can log this > information from onLogin() method of an Ftplet, but then I would not have > any information on why the login has actually failed. All I've is - 530 > Authentication Failed reply. > > Another benefit would be if I want to implement my user manager based on > user name and IP address. For example let User1 login if and only if he is > connecting from IP address xxx.xxx.xxx.xxx. Not sure if any one does this > kind of authentication, but in case if some one want to, this change should > help. > > More info about this feature request can be found in the thread - > http://www.mail-archive.com/dev@mina.apache.org/msg12942.html. > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > >