[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mina-dev
Subject:    [jira] Commented: (FTPSERVER-399) DefaultFtpHandler.sessionOpened
From:       "Niklas Gustavsson (JIRA)" <jira () apache ! org>
Date:       2011-02-24 20:33:38
Message-ID: 185427314.14337.1298579618525.JavaMail.tomcat () hel ! zones ! apache ! org
[Download RAW message or body]


    [ https://issues.apache.org/jira/browse/FTPSERVER-399?page=com.atlassian.jira.plug \
in.system.issuetabpanels:comment-tabpanel&focusedCommentId=12999027#comment-12999027 \
] 

Niklas Gustavsson commented on FTPSERVER-399:
---------------------------------------------

This is intentional to keep backwards compatibility with earlier versions of \
FtpServer (before the Ftplet class hierarchy was changed to introduce \
DefaultFtpHandler. That said, I'm all for changing this in 1.1.0, but not in 1.0.6.

> DefaultFtpHandler.sessionOpened does not check if the Ftplet returned a \
>                 FtpletResult.SKIP
> -----------------------------------------------------------------------------------------
>  
> Key: FTPSERVER-399
> URL: https://issues.apache.org/jira/browse/FTPSERVER-399
> Project: FtpServer
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.0.5
> Reporter: Sai Pullabhotla
> Assignee: Sai Pullabhotla
> Fix For: 1.1.0
> 
> 
> I was trying to use an Ftplet to send a custom welcome message using the \
> Ftplet.onConnect. Within this method, I write a 220 reply to the client with my \
> custom message, and return FtpletResult.SKIP. I expect that the server would not \
> send the default welcome message, but it does it any how. When I looked at the \
> server code, the DefaultFtpHandler does not handle the FtpletResult.SKIP case. I \
> think it needs to be updated so a welcome message is sent only if the Ftplet \
> returned DEFAULT result. 

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic