commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rory Winston (JIRA)" <>
Subject [jira] Closed: (NET-36) [net] PATCH] FTP and FTPClient changes
Date Wed, 20 Dec 2006 10:14:29 GMT
     [ ]

Rory Winston closed NET-36.

> [net] PATCH] FTP and FTPClient changes
> --------------------------------------
>                 Key: NET-36
>                 URL:
>             Project: Commons Net
>          Issue Type: Bug
>    Affects Versions: 1.2 Final
>         Environment: Operating System: other
> Platform: Other
>            Reporter: Joseph Hindsley
>         Attachments:,,,
> 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:
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message