commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Holger Rehn (JIRA)" <>
Subject [jira] [Created] (NET-583) FTPClient.getReplyString() returns wrong value after connect()
Date Fri, 27 Nov 2015 09:18:11 GMT
Holger Rehn created NET-583:

             Summary: FTPClient.getReplyString() returns wrong value after connect()
                 Key: NET-583
             Project: Commons Net
          Issue Type: Bug
          Components: FTP
    Affects Versions: 3.3
            Reporter: Holger Rehn

If the FTPClient's automatic server encoding detection is enabled, a FEAT command is issued
in method _connectAction_() [indirectly via hasFeature(String)]. After that, the _replyCode
and _replyLines fields are stored back to their previous values in _connectAction_(), but
the _newReplyString flag isn't set to true. Because of that, you will then get back the reply
to the FEAT command from getReplyString(), instead of the server's welcome message. Furthermore,
you may get back a reply code that doesn't match that reply string. We have encountered a
case when we got back reply code 220 after FTPClient.connect(), but reply string was "530
Not logged in.".

This error can easily be fixed by adding the following line to around line
_newReplyString = true;

--- src/org/apache/commons/net/ftp/
+++ src/org/apache/commons/net/ftp/	(working copy)
@@ -941,6 +941,7 @@
            _replyCode = oldReplyCode;
+           _newReplyString = true;

This message was sent by Atlassian JIRA

View raw message