[
https://issues.apache.org/jira/browse/NET-291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12763301#action_12763301
]
Joseph Boyd commented on NET-291:
---------------------------------
This same behavior, calling __initDefaults() inside connectAction, also resets
the fileType to its default, even though the javadocs for
FTPClient.setFileType() indicate the type should stay the same until reset
> enterLocalPassiveMode is set back to Active on connect
> ------------------------------------------------------
>
> Key: NET-291
> URL: https://issues.apache.org/jira/browse/NET-291
> Project: Commons Net
> Issue Type: Bug
> Affects Versions: 2.0
> Reporter: Kevin Brown
>
> enterLocalPassiveMode (exhibit A) docs claim that mode will be set to
> PASSIVE_LOCAL_DATA_CONNECTION_MODE "some other method such as"
> enterLocalActiveMode is called (exhibit B). However, active mode is being
> entered after connect is called. This behavior can be easily observed by
> modifying FtpExample by moving ftp.enterLocalPassiveMode() to before
> ftp.connect(server).
> Perhaps either the code or docs could be updated to remedy this. Versions
> prior to 2.0 behaved as documented.
>
> exhibit A:
> /***
> * Set the current data connection mode to
> * <code> PASSIVE_LOCAL_DATA_CONNECTION_MODE </code>. Use this
> * method only for data transfers between the client and server.
> * This method causes a PASV command to be issued to the server
> * before the opening of every data connection, telling the server to
> * open a data port to which the client will connect to conduct
> * data transfers. The FTPClient will stay in
> * <code> PASSIVE_LOCAL_DATA_CONNECTION_MODE </code> until the
> * mode is changed by calling some other method such as
> * {...@link #enterLocalActiveMode enterLocalActiveMode() }
> ***/
> public void enterLocalPassiveMode()
> {
> __dataConnectionMode = PASSIVE_LOCAL_DATA_CONNECTION_MODE;
> // These will be set when just before a data connection is opened
> // in _openDataConnection_()
> __passiveHost = null;
> __passivePort = -1;
> }
> exhibit B
> @Override
> protected void _connectAction_() throws IOException
> {
> super._connectAction_();
> __initDefaults();
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
