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

List:       mina-dev
Subject:    [jira] Created: (FTPSERVER-209) onSite() method of DefaultFtplet
From:       "Gary Bell (JIRA)" <jira () apache ! org>
Date:       2008-10-30 15:03:44
Message-ID: 114893350.1225379024239.JavaMail.jira () brutus
[Download RAW message or body]

onSite() method of DefaultFtplet called after command has been processed
------------------------------------------------------------------------

                 Key: FTPSERVER-209
                 URL: https://issues.apache.org/jira/browse/FTPSERVER-209
             Project: FtpServer
          Issue Type: Bug
          Components: Ftplets
    Affects Versions: 1.0-M3
         Environment: Windows XP, Sun JVM 1.5.0
            Reporter: Gary Bell
            Priority: Minor


According to the ftplet API documentation, the onSite() method is described thus:

-- PASTE --
This is SITE command start notification method. It gives a chance to implement custom \
SITE command. If this method returns SKIP or DISCONNECT, the existing SITE commands \
                will not be executed.
-- PASTE --

But the onSite() method is only called inside the DefaultFtplet.afterCommand() method \
which means that the custom SITE command issued by the user always tries to be \
processed by the server, resulting in a "502 - Command SITE not implemented for xxx"  \
message being displayed to the user even though the custom command is executed. \
Furthermore, returning FTPletResult.SKIP after processing the custom code to prevent \
further processing of the request  by the server has no effect. For the onSite() \
method to work as per the documentation, it would have to be called in the \
beforeCommand() function.

I can create sample code for clarification if you like.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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