net-http-persistent 2.9 → 2.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ea7002c40a540ea921855f2b04c1862c0e501bb9
-  data.tar.gz: 2e542b3cbdc4752799409f8dd1ecbbbb1ee84f0a
+  metadata.gz: 55d2e4859498dd8d729452db6118d77d13535bd1
+  data.tar.gz: 7d1a5487a20e2d8d44be10241010e7022af2262f
 SHA512:
-  metadata.gz: 591fb46b7ac365d1268935f44a449cdd4cfc56c64398d3e258baf7cd5e215fd213e42f5f9fcc5fdc032daae2dc93a34404833dd3a9afc6dc0ed4828547709f11
-  data.tar.gz: 3865436c02a8efecc81c335bb603d1ca2b540a3dde995df17f7dc844e38d0715279a02d4cf77128bff9f837380530e3d3487e3a396e05a6e1fa4936b904ea7cf
+  metadata.gz: 736f79380094d6b6a84b923c6e8251ee75e95b66185bb072ba7f19450263c0e96b95e0ce92622050cddb3a5dc8e9e2fc3c04ab82f1787e3524b22902f340d9b9
+  data.tar.gz: 0b0c2b885b275bb98e64e22d4c96869496ff3602e17c9be2b846015293354646d4685204d7b32ed56d79a55dd9633fd84bb8b09fc49fb83448cae72a849dd9e4

data/History.txt

@@ -1,8 +1,18 @@
+=== 2.9.1 / 2014-01-22
+
+* Bug fixes
+  * Added license to gemspec.  Issue #47 by Benjamin Fleischer
+  * Set Net::HTTP#keep_alive_timeout when supported by ruby.  Pull request #53
+    by Dylan Thacker-Smith.
+  * The backtrace is preserved for errors in #reset to help with debugging.
+    Issue #41 by Andrew Cholakian.
+
 === 2.9 / 2013-07-24
 
 * Minor enhancement
-  * Added max_requests to avoid ECONNRESET for a server that allows a limited
-    number of requests on a connection.  Pull request #42 by James Tucker.
+  * Added Net::HTTP::Persistent#max_requests to avoid ECONNRESET for a server
+    that allows a limited number of requests on a connection.  Pull request
+    #42 by James Tucker.
   * Request failures are now raised with the backtrace of the original
     exception.  This gives better insight into the reason for the failure.
     See #41 by Andrew Cholakian.
@@ -17,8 +27,8 @@
   * Requests retried by ruby 2.x are no longer retried by net-http-persistent.
   * Finish the connection if an otherwise unhandled exception happens during a
     request.  Bug #46 by Mark Oude Veldhuis.
-  * detect_idle_timeout now assumes a StandardError indicates the idle timeout
-    has been found.  Bug #43 by James Tucker.
+  * Net::HTTP::Persistent::detect_idle_timeout now assumes a StandardError
+    indicates the idle timeout has been found.  Bug #43 by James Tucker.
 
 === 2.8 / 2012-10-17
 

data/Rakefile

@@ -7,14 +7,18 @@ Hoe.plugin :git
 Hoe.plugin :minitest
 Hoe.plugin :travis
 
-Hoe.spec 'net-http-persistent' do |p|
+Hoe.spec 'net-http-persistent' do
   developer 'Eric Hodel', 'drbrain@segment7.net'
 
   self.readme_file      = 'README.rdoc'
   self.extra_rdoc_files += Dir['*.rdoc']
 
+  license 'MIT'
+
   rdoc_locations <<
     'docs.seattlerb.org:/data/www/docs.seattlerb.org/net-http-persistent/'
+
+  dependency 'minitest', '~> 5.2', :development
 end
 
 # vim: syntax=Ruby

data/lib/net/http/persistent.rb

@@ -203,7 +203,7 @@ class Net::HTTP::Persistent
   ##
   # The version of Net::HTTP::Persistent you are using
 
-  VERSION = '2.9'
+  VERSION = '2.9.1'
 
   ##
   # Exceptions rescued for automatic retry on ruby 2.0.0.  This overlaps with
@@ -631,6 +631,7 @@ class Net::HTTP::Persistent
     start connection unless connection.started?
 
     connection.read_timeout = @read_timeout if @read_timeout
+    connection.keep_alive_timeout = @idle_timeout if @idle_timeout && connection.respond_to?(:keep_alive_timeout=)
 
     connection
   rescue Errno::ECONNREFUSED
@@ -955,9 +956,13 @@ class Net::HTTP::Persistent
 
     start connection
   rescue Errno::ECONNREFUSED
-    raise Error, "connection refused: #{connection.address}:#{connection.port}"
+    e = Error.new "connection refused: #{connection.address}:#{connection.port}"
+    e.set_backtrace $@
+    raise e
   rescue Errno::EHOSTDOWN
-    raise Error, "host down: #{connection.address}:#{connection.port}"
+    e = Error.new "host down: #{connection.address}:#{connection.port}"
+    e.set_backtrace $@
+    raise e
   end
 
   ##

data/test/test_net_http_persistent.rb

@@ -274,6 +274,7 @@ class TestNetHttpPersistent < Minitest::Test
   def test_connection_for
     @http.open_timeout = 123
     @http.read_timeout = 321
+    @http.idle_timeout = 42
     c = @http.connection_for @uri
 
     assert_kind_of @http_class, c
@@ -283,6 +284,7 @@ class TestNetHttpPersistent < Minitest::Test
 
     assert_equal 123, c.open_timeout
     assert_equal 321, c.read_timeout
+    assert_equal 42, c.keep_alive_timeout unless RUBY_1
 
     assert_includes conns[0].keys, 'example.com:80'
     assert_same c, conns[0]['example.com:80']
@@ -1337,6 +1339,7 @@ class TestNetHttpPersistent < Minitest::Test
     end
 
     assert_match %r%host down%, e.message
+    assert_match __FILE__, e.backtrace.first
   end
 
   def test_reset_io_error
@@ -1361,6 +1364,7 @@ class TestNetHttpPersistent < Minitest::Test
     end
 
     assert_match %r%connection refused%, e.message
+    assert_match __FILE__, e.backtrace.first
   end
 
   def test_retry_change_requests_equals

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: net-http-persistent
 version: !ruby/object:Gem::Version
-  version: '2.9'
+  version: 2.9.1
 platform: ruby
 authors:
 - Eric Hodel
@@ -30,50 +30,50 @@ cert_chain:
   KDyY1VIazVgoC8XvR4h/95/iScPiuglzA+DBG1hip1xScAtw05BrXyUNrc9CEMYU
   wgF94UVoHRp6ywo8I7NP3HcwFQDFNEZPNGXsng==
   -----END CERTIFICATE-----
-date: 2013-07-24 00:00:00.000000000 Z
+date: 2014-01-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
 - !ruby/object:Gem::Dependency
   name: hoe
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '3.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '3.7'
 description: |-
   Manages persistent connections using Net::HTTP plus a speed fix for Ruby 1.8.
   It's thread-safe too!
@@ -94,8 +94,8 @@ extra_rdoc_files:
 - Manifest.txt
 - README.rdoc
 files:
-- .autotest
-- .gemtest
+- ".autotest"
+- ".gemtest"
 - History.txt
 - Manifest.txt
 - README.rdoc
@@ -106,27 +106,28 @@ files:
 - test/test_net_http_persistent.rb
 - test/test_net_http_persistent_ssl_reuse.rb
 homepage: http://docs.seattlerb.org/net-http-persistent
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options:
-- --main
+- "--main"
 - README.rdoc
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: net-http-persistent
-rubygems_version: 2.0.4
+rubygems_version: 2.2.1
 signing_key: 
 specification_version: 4
 summary: Manages persistent connections using Net::HTTP plus a speed fix for Ruby