simo385 commented on pull request #51: URL: https://github.com/apache/commons-net/pull/51#issuecomment-732211772
Dear @garydgregory, yes, currently, the workaround is to call _org.apache.commons.net.SocketClient.setConnectTimeout(int)_, but I've had a lot of problem in my production environment (multi-threading) because of infinity timeout. Also, I've spent a lot of time to find out the issue. I understand your worry, we can discuss the default timeout value, but IMHO to set a default timeout = infinity is really dangerous, especially in multi-threading environment. Many thanks for your support. Simone ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
