diff options
author | nagachika <nagachika@ruby-lang.org> | 2022-10-22 17:08:57 +0900 |
---|---|---|
committer | nagachika <nagachika@ruby-lang.org> | 2022-10-22 17:08:57 +0900 |
commit | 2c1b1eae7c984c9ff4788d382bc2edcd4ed91ef5 (patch) | |
tree | 2398291f0e38a55977114e1b3bb8f5eb6153e158 | |
parent | 3f3aebc23729f53c8c3564a33e7b83fc02072449 (diff) | |
download | ruby-2c1b1eae7c984c9ff4788d382bc2edcd4ed91ef5.tar.gz |
merge revision(s) f88bff770578583a708093f4a0d8b1483a1d2039:
[ruby/net-http] Revert "Replace Timeout.timeout in Net:HTTP#connect"
This reverts commit https://github.com/ruby/net-http/commit/753cae3bbccc.
https://github.com/ruby/net-http/commit/98caa38204
---
lib/net/http.rb | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
-rw-r--r-- | lib/net/http.rb | 15 | ||||
-rw-r--r-- | version.h | 2 |
2 files changed, 9 insertions, 8 deletions
diff --git a/lib/net/http.rb b/lib/net/http.rb index f004bce836..002e3cf10e 100644 --- a/lib/net/http.rb +++ b/lib/net/http.rb @@ -994,13 +994,14 @@ module Net #:nodoc: end debug "opening connection to #{conn_addr}:#{conn_port}..." - begin - s = Socket.tcp conn_addr, conn_port, @local_host, @local_port, connect_timeout: @open_timeout - rescue => e - e = Net::OpenTimeout.new(e) if e.is_a?(Errno::ETIMEDOUT) #for compatibility with previous versions - raise e, "Failed to open TCP connection to " + - "#{conn_addr}:#{conn_port} (#{e.message})" - end + s = Timeout.timeout(@open_timeout, Net::OpenTimeout) { + begin + TCPSocket.open(conn_addr, conn_port, @local_host, @local_port) + rescue => e + raise e, "Failed to open TCP connection to " + + "#{conn_addr}:#{conn_port} (#{e.message})" + end + } s.setsockopt(Socket::IPPROTO_TCP, Socket::TCP_NODELAY, 1) debug "opened" if use_ssl? @@ -11,7 +11,7 @@ # define RUBY_VERSION_MINOR RUBY_API_VERSION_MINOR #define RUBY_VERSION_TEENY 3 #define RUBY_RELEASE_DATE RUBY_RELEASE_YEAR_STR"-"RUBY_RELEASE_MONTH_STR"-"RUBY_RELEASE_DAY_STR -#define RUBY_PATCHLEVEL 166 +#define RUBY_PATCHLEVEL 167 #define RUBY_RELEASE_YEAR 2022 #define RUBY_RELEASE_MONTH 10 |