Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <956106366.174051263268014446.JavaMail.jira@brutus.apache.org>
Date: Tue, 12 Jan 2010 03:46:54 +0000 (UTC)
From: "Stephane Godin (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Created: (NET-307) One of the "connect" method in class
 org.apache.commons.net.SocketClient doesn't handle connection timeout
 properly
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

One of the "connect" method in class org.apache.commons.net.SocketClient doesn't handle connection timeout properly
-------------------------------------------------------------------------------------------------------------------

                 Key: NET-307
                 URL: https://issues.apache.org/jira/browse/NET-307
             Project: Commons Net
          Issue Type: Improvement
    Affects Versions: 2.0
         Environment: Not applicable
            Reporter: Stephane Godin


org.apache.commons.net.SocketClient method connect (line 225) doesn't use the configured timeout value.

Original code:

{code:title=SocketClient.java (line 225)|borderStyle=solid}
    public void connect(String hostname, int port,
                        InetAddress localAddr, int localPort)
    throws SocketException, IOException
    {
        _socket_ = _socketFactory_.createSocket(hostname, port, localAddr, localPort);
        _connectAction_();
    }
{code}

Working code:

{code:title=SocketClient.java (line 225)|borderStyle=solid}
    public void connect(String hostname, int port,
                        InetAddress localAddr, int localPort)
    throws SocketException, IOException
    {
        _socket_ = _socketFactory_.createSocket();
        _socket_.connect(new InetSocketAddress(hostname, port), connectTimeout);
        _connectAction_();
    }
{code}


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.