What do you all think of this RFE? I would be happy to add the onLoginFail() method to the FtpLet interface if you don't disagree.

I would be more reluctant to add in the IP in the UserManager, that doesn't seem like the right place to allow users to extend functionality for handling an event like this.

/niklas

Clinton Foster (JIRA) wrote:
Add onLoginFail() to Ftplet interface
-------------------------------------

                 Key: FTPSERVER-52
                 URL: http://issues.apache.org/jira/browse/FTPSERVER-52
             Project: FtpServer
          Issue Type: Improvement
            Reporter: Clinton Foster
            Priority: Minor


Particularly in cases where the FTP server is embedded in another application, 
it is important to be able to determine the IP address of the remote client 
when authentication fails. One possibility would be to provide access to the 
remote address from UserManager.authenticate(). (This might be useful for other 
reasons as well.) A more generic approach that would not require a custom 
UserManager implementation would be to add an onLoginFail() method to the 
Ftplet interface. I think both enhancements would be useful, although if only 
one or the other could be done I would favor adding Ftplet.onLoginFail().


Reply via email to