[NET-399] ftp data connection does not use connectTimeout

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0, 2.2
• Fix Version/s: 3.0
• Component/s: FTP
• Assignee: Unassigned
• Reporter: Noah Levitt
• Created: 04/Apr/11 20:06
• Updated: 04/Jun/11 19:26
• Resolved: 05/Apr/11 22:22

Description

Deep inside the lengthy method org.apache.commons.net.ftp.FTPClient.openDataConnection(int, String) we find this line

socket = socketFactory.createSocket(__passiveHost, __passivePort);

Which I believe does not set the connect timeout when opening the connection. If the line were changed to the following, it would use the connectTimeout property.

socket = connect(__passiveHost, __passivePort)