commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rory Winston (JIRA)" <j...@apache.org>
Subject [jira] Closed: (NET-36) [net] PATCH] FTP and FTPClient changes
Date Wed, 20 Dec 2006 10:14:29 GMT
     [ http://issues.apache.org/jira/browse/NET-36?page=all ]

Rory Winston closed NET-36.
---------------------------


> [net] PATCH] FTP and FTPClient changes
> --------------------------------------
>
>                 Key: NET-36
>                 URL: http://issues.apache.org/jira/browse/NET-36
>             Project: Commons Net
>          Issue Type: Bug
>    Affects Versions: 1.2 Final
>         Environment: Operating System: other
> Platform: Other
>            Reporter: Joseph Hindsley
>         Attachments: FTP.java.patch.txt, FTPClient.java.patch.txt, FTPTest.java.patch.txt,
FTPTest.java.patch.txt
>
>
> I've included 3 patch files for changes I've made to the FTP andFTPClient
> classes in the commons-net package.
> The first patch is for the FTP class making it extend SocketClientinstead of
> TelnetClient. I noticed that the behavior of theTelnetClient's input stream
> reader thread was effectively ignoring thesocket's SOTimeout causing reads to
> hang forever if the server decidednot to respond to a client request at all.
> This should also answer oneof the goals from the TODO list:
> "Divorce FTPClient from TelnetClient, getting rid of the TelnetClientthreads
> which cause problems on some platforms (e.g., MacOS)."
> The second patch is for an FTPTest unit test. I've covered most of thebasic
> methods (connect(), disconnect(), sendCommand(), getReplyCode(),etc). Ignored
> for now are the convenience methods since they all callsendCommand() underneath.
> Part of the FTPTest class is a DummyFTPServerinner class which is used to
> communicate to the test FTP class - don'tknow if that would be useful elsewhere
> (maybe part of FTPClient unittests), so you might consider making it a utility
> class for other unittests.
> Finally I've attached a patch for minor changes to FTPClient:
>  - changed __storeFile() from private to protected so that it can beused by
> classes that extend FTPClient
>  - added __storeFile(String, String) method - so that the commands itaccepts are
> not limited to what's found in FTPCommand. Note: the__storeFile(int, String)
> method now calls the __storeFile(String,String) method.
>  - added _openDataConnection_(String, String) method - so that thecommands it
> accepts are not limited to what's found in FTPCommand. Note:the
> _openDataConnection_(int, String) method now calls
> the_openDataConnection_(String, String) method.
> Hopefully you'll find the changes agreeable and will incorporate theminto the
> code base.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message