lib/net/http.rb: use connect_timeout instead of Timeout

lib/net/pop.rb: ditto

lib/net/ftp.rb: ditto

lib/net/smtp.rb: ditto

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@66659 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

lib/net/ftp.rb

@@ -329,14 +329,18 @@ module Net
     # SOCKS_SERVER, then a SOCKSSocket is returned, else a Socket is
     # returned.
     def open_socket(host, port) # :nodoc:
-      return Timeout.timeout(@open_timeout, OpenTimeout) {
+      if defined? SOCKSSocket and ENV["SOCKS_SERVER"]
-        if defined? SOCKSSocket and ENV["SOCKS_SERVER"]
+        @passive = true
-          @passive = true
+        return Timeout.timeout(@open_timeout, OpenTimeout) {
           SOCKSSocket.open(host, port)
-        else
+        }
-          Socket.tcp(host, port)
+      else
+        begin
+          return Socket.tcp(host, port, connect_timeout: @open_timeout)
+        rescue Errno::ETIMEDOUT
+          raise OpenTimeout, "execution expired"
         end
-      }
+      end
     end
     private :open_socket
 

lib/net/http.rb

@@ -942,14 +942,15 @@ module Net   #:nodoc:
       end
 
       D "opening connection to #{conn_address}:#{conn_port}..."
-      s = Timeout.timeout(@open_timeout, Net::OpenTimeout) {
+      begin
-        begin
+        s = Socket.tcp(conn_address, conn_port, @local_host, @local_port, connect_timeout: @open_timeout)
-          TCPSocket.open(conn_address, conn_port, @local_host, @local_port)
+      rescue Errno::ETIMEDOUT => e
-        rescue => e
+        raise Net::OpenTimeout, "Failed to open TCP connection to " +
-          raise e, "Failed to open TCP connection to " +
+          "#{conn_address}:#{conn_port} (#{e.message})"
-            "#{conn_address}:#{conn_port} (#{e.message})"
+      rescue => e
-        end
+        raise e, "Failed to open TCP connection to " +
-      }
+          "#{conn_address}:#{conn_port} (#{e.message})"
+      end
       s.setsockopt(Socket::IPPROTO_TCP, Socket::TCP_NODELAY, 1)
       D "opened"
       if use_ssl?

lib/net/pop.rb

@@ -541,8 +541,10 @@ module Net
 
     # internal method for Net::POP3.start
     def do_start(account, password) # :nodoc:
-      s = Timeout.timeout(@open_timeout, Net::OpenTimeout) do
+      begin
-        TCPSocket.open(@address, port)
+        s = Socket.tcp(@address, port, connect_timeout: @open_timeout)
+      rescue Errno::ETIMEDOUT
+        raise Net::OpenTimeout, "execution expired"
       end
       if use_ssl?
         raise 'openssl library not installed' unless defined?(OpenSSL)

lib/net/smtp.rb

@@ -545,8 +545,10 @@ module Net
         check_auth_method(authtype || DEFAULT_AUTH_TYPE)
         check_auth_args user, secret
       end
-      s = Timeout.timeout(@open_timeout, Net::OpenTimeout) do
+      begin
-        tcp_socket(@address, @port)
+        s = Socket.tcp(@address, @port, connect_timeout: @open_timeout)
+      rescue Errno::ETIMEDOUT
+        raise Net::OpenTimeout, "execution expired"
       end
       logging "Connection opened: #{@address}:#{@port}"
       @socket = new_internet_message_io(tls? ? tlsconnect(s) : s)