fetch-api 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff43ec8ec36d621acb463b7327146e89df97446714df7747626276bb51ff4c18
-  data.tar.gz: de99e42e871912c41df6a5afabb1200f662d5aeba7b6659a4483434c8ddca3c7
+  metadata.gz: 4217a0ddb4ec43da71a630ba5de24d9a6a02af0f5cf2b39286702c1cc512c699
+  data.tar.gz: 05abc4f266fab3577c2223de54957b8c6a84e7b3ccc24df428f40e771c1e311d
 SHA512:
-  metadata.gz: 7b56ad7b6ac2788343928c4686f48de71e1dc57491e9e13efa5cc479f4f7f57af9d4efad3e7a79cf75fd287d683bb21ec053b69de0c66cdd39a0aef571f86c7b
-  data.tar.gz: 8895b9835e6637a9d4cd845708e2a133b2f6239baab1729478f16ae9f25ea3df927d678c41ebfbcc26cb3482a43c5fb616779df89a21a8a1aa634f0e18ebe392
+  metadata.gz: c7caa0c96f93f03ae799a121fc7f25410f401c9c192a7546f97627c760cb638627b79d01b0911ed8a4f2ebaacdaa619e86e84a2b88443a721f066a4a9fd4c1d9
+  data.tar.gz: 826e3c4ff75b7b3c3f0886949a6fe141cdd360fd31da86b42b08aabde98b0a608ac6f51561b65819b65622a1a2f344f5e6784505bafb990d0aad300ac8f378b6

data/README.md

@@ -1,8 +1,8 @@
-# Fetch API for Ruby
+# Fetch::API
 
 Ruby's Net::HTTP is very powerful, but has a complicated API. OpenURI is easy to use, but has limited functionality. Third-party HTTP clients each have different APIs, and it can sometimes be difficult to learn how to use them.
 
-There is one HTTP client that we can all use. It is the browser's Fetch API. This is an HTTP client for Ruby that can be used in a way that is similar to the Fetch API.
+There is one HTTP client that we can all use. It is the browser's Fetch API. Fetch::API is an HTTP client for Ruby that can be used in a way that is similar to the Fetch API.
 
 This is not intended to be a complete copy of the Fetch API. For example, responses are returned synchronously rather than asynchronously, as that is more familiar behavior for Ruby programmers. It is only "like" the Fetch API.
 
@@ -102,16 +102,16 @@ res = fetch('http://example.com', **{
 })
 ```
 
-## Connection pooling
+### Connection pooling
 
-Fetch API automatically pools and reuses connections to the same origin. Connections are closed after a certain amount of time has passed since the last use, and a new connection is used the next time. Different connections are returned for different threads (in other words, Fetch API is thread-safe).
+Fetch::API automatically pools and reuses connections to the same origin. Connections are closed after a certain amount of time has passed since the last use, and a new connection is used the next time. Different connections are returned for different threads (in other words, Fetch::API is thread-safe).
 
-These values (in seconds) can be configured as follows:
+These values can be configured as follows (in seconds):
 
 ``` ruby
 Fetch.configure do |config|
-  config.connection_max_idle_time = 10 # default
-  config.keep_alive_timeout       = 2  # default
+  config.max_idle_time      = 10 # default
+  config.keep_alive_timeout = 2  # default
 end
 ```
 

data/lib/fetch/client.rb

@@ -48,7 +48,10 @@ module Fetch
       when Net::HTTPRedirection
         case redirect.to_s
         when 'follow'
-          fetch(res['Location'], method:, headers:, body:, redirect:, _redirected: true) # steep:ignore ArgumentTypeMismatch
+          # @type var location: String
+          location = res['Location']
+
+          fetch(location, method:, headers:, body:, redirect:, _redirected: true) # steep:ignore ArgumentTypeMismatch
         when 'error'
           raise RedirectError, "redirected to #{res['Location']}"
         when 'manual'
@@ -67,7 +70,7 @@ module Fetch
       if pool = Thread.current.thread_variable_get(:fetch_connection_pool)
         pool
       else
-        Thread.current.thread_variable_set :fetch_connection_pool, ConnectionPool.new
+        Thread.current.thread_variable_set(:fetch_connection_pool, ConnectionPool.new)
       end
     end
 

data/lib/fetch/config.rb

@@ -1,3 +1,3 @@
 module Fetch
-  Config = Struct.new(:connection_max_idle_time, :keep_alive_timeout)
+  Config = Struct.new(:max_idle_time, :keep_alive_timeout)
 end

data/lib/fetch/connection_pool.rb

@@ -13,7 +13,6 @@ module Fetch
       @sweeper = Thread.new {
         loop do
           sleep 1
-
           sweep
 
           Thread.stop if @connections.empty?
@@ -42,13 +41,16 @@ module Fetch
 
           entry.connection
         else
-          Net::HTTP.new(uri.host, uri.port).tap {|http| # steep:ignore ArgumentTypeMismatch
+          # @type var host: String
+          host = uri.host
+
+          Net::HTTP.new(host, uri.port).tap {|http|
             http.use_ssl            = uri.scheme == 'https'
             http.keep_alive_timeout = Fetch.config.keep_alive_timeout
 
-            @connections[uri.origin] = Entry.new(connection: http, in_use: true)
-
             http.start
+
+            @connections[uri.origin] = Entry.new(connection: http, in_use: true)
           }
         end
       }.tap {
@@ -70,7 +72,7 @@ module Fetch
         @connections.each do |origin, entry|
           next if entry.in_use
 
-          if entry.last_used + Fetch.config.connection_max_idle_time < Time.now
+          if entry.last_used + Fetch.config.max_idle_time < Time.now
             entry.connection.finish
 
             @connections.delete origin

data/lib/fetch/version.rb

@@ -1,3 +1,3 @@
 module Fetch
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

data/lib/fetch.rb

@@ -10,7 +10,7 @@ module Fetch
   end
 
   configure do |config|
-    config.connection_max_idle_time = 10
-    config.keep_alive_timeout       = 2
+    config.max_idle_time      = 10
+    config.keep_alive_timeout = 2
   end
 end

data/sig/fetch/config.rbs

@@ -1,6 +1,6 @@
 module Fetch
   class Config
-    attr_accessor connection_max_idle_time: Integer
+    attr_accessor max_idle_time: Integer
     attr_accessor keep_alive_timeout: Integer
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fetch-api
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Keita Urashima