commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Commented] (NET-472) FTPClient doesn't remember __dataConnectionMode
Date Sat, 18 Aug 2012 09:08:38 GMT


Sebb commented on NET-472:

The same consideration applies to some other variables that have setters:

__fileType           = FTP.ASCII_FILE_TYPE;
__fileStructure      = FTP.FILE_STRUCTURE;
__fileFormat         = FTP.NON_PRINT_TEXT_FORMAT;
__fileTransferMode   = FTP.STREAM_TRANSFER_MODE;
__bufferSize         = Util.DEFAULT_COPY_BUFFER_SIZE;

There does not seem to be any particular reason to initialise them anywhere except the ctor
(or in the variable declarations)

__initDefaults() should probably be reserved for items that must be reset when the connection
state changes, for example __systemName is a cached value that is connection-specific.
> FTPClient doesn't remember __dataConnectionMode
> -----------------------------------------------
>                 Key: NET-472
>                 URL:
>             Project: Commons Net
>          Issue Type: Bug
>          Components: FTP
>            Reporter: Yuliya Polyeno
>            Priority: Minor
> There is an issue with FTPClient.
> 1) When you create FTPClient in constructor you invoke __initDefaults() which would set
> 2) if you try after creation set up __dataConnectionMode into PASSIVE_LOCAL_DATA_CONNECTION_MODE
it will be overridden when you would invoke connect() method because connect method again
will invoke __initDefaults() which will override my setup, the same will happen when you disconect()
because it will invoke __initDefaults() again.
> I believe this is wrong behavior I don't need to switch on passive mode every time after
connect. It should be only done when I create FTPClient.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message