You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by "Rory Winston (JIRA)" <ji...@apache.org> on 2006/12/20 11:14:29 UTC
[jira] Closed: (NET-36) [net] PATCH] FTP and FTPClient changes
[ 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