http.rb 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 448e7dce1f8dc9a0b23a659b351d10e241ccf1e3cf853663046ec4d7e6e0c8fe
-  data.tar.gz: 75123ecd90c4f5333b8c9a6d62435ab20b46623a5871eb8464a224a92c900ed1
+  metadata.gz: 06d8c18c5bfecb35e07a63571d9f4a874da28ae9c20a368781e2ecb01f3ebf7f
+  data.tar.gz: f2f58d2f70bb3aeaa7b03d7a2ad3c234fe23536a0e53a8dec312b22da8f0f695
 SHA512:
-  metadata.gz: 7fadfa5b0e33af922edc397877c6fecdff9fd0e5fdb2c8bbb17b9aaeb33f7cddc61553129e1ae6dafc7ee7367a4e1e0a71fa380119020dafbb50e66873f2333c
-  data.tar.gz: d5b35c43b60b13b3e34b1b56a5d02189266401ccb3a9d63b9d87ead07330f96efb80e2577391eea8ed0ec2a734050d7b22aa0fb21e50cfe334b53ca700ede7a6
+  metadata.gz: 989858d90f01ab5b85678baf5241ff61a1e620a3c338442dd8ba512d11c5de81fa6cdb92abb39431e88d4e4372c9df4cd11883d06d111bc7a7e3bec91c1c8b55
+  data.tar.gz: ace86800c9010db5b30b6d6814196ba0d689781757076f0fab6b1b0c0b235d2557e350180010b20e6e0face72d55eb2a2c8c9933ebaed51b9ec2c49a3f4b3fc7

data/CHANGELOG

@@ -1,6 +1,35 @@
 # CHANGELOG
 
-## 20260609
+## 20260610
+
+1.3.0: Rename VERBS to METHODS; VERBS retained as a deprecated alias.
+
+RFC 9110 calls these "methods", not "verbs". HTTP::VERBS becomes HTTP::METHODS and HTTP::RETRY::VERBS becomes HTTP::RETRY::METHODS, with the old names kept as deprecated aliases (via deprecate_constant) so nothing breaks. The call surface — the named methods, the options hash, HTTP.request, request_with_body/request_without_body — is unchanged. The aliases are scheduled for removal in 2.0.0; that removal is the breaking change the major bump is reserved for.
+
+1. /lib/HTTP/verbs.rb/lib/HTTP/METHODS.rb/: the file is now named for the METHODS module it defines, matching RETRY.rb and VERSION.rb.
+2. ~ HTTP/METHODS.rb: /VERBS/METHODS/ for the WITHOUT_BODY/WITH_BODY container; + VERBS deprecated alias; /verb/request_method/ in the block variable.
+3. ~ HTTP.rb: require 'HTTP/METHODS' in place of require 'HTTP/verbs'.
+4. ~ HTTP/RETRY.rb: /VERBS/METHODS/ for the default retry method list; + VERBS deprecated alias. retry_config now accepts the :retry_methods option (preferred) and still honours :retry_verbs as a backwards-compatible alias; its internal config key is /verbs/methods/.
+5. ~ HTTP/request.rb: the 307/308 redirect handler now references METHODS::WITH_BODY, and the retry guard reads config[:methods]; /verb/request_method/.
+6. ~ test/HTTP/RETRY_test.rb: + retry_methods opt-in and retry_methods-over-retry_verbs precedence cases; the option-leak test now covers both :retry_methods and :retry_verbs.
+7. + test/HTTP/METHODS_test.rb
+8. ~ README.md: retry_methods documented as the option key, with retry_verbs noted as still accepted; the retry default is shown as HTTP::RETRY::METHODS.
+9. ~ HTTP::VERSION: /1.2.1/1.3.0/
+10. ~ CHANGELOG: + 1.3.0 entry
+
+## 20260610
+
+1.2.1: Split the request builders into their own files.
+
+request_with_body and request_without_body, added in 1.2.0, each move into their own file, matching the one-method-per-file layout used throughout lib/. Purely internal reorganisation — the methods, their signatures, and their behaviour are unchanged — so it is a patch, not a minor.
+
+1. + lib/HTTP/request_without_body.rb: HTTP.request_without_body, extracted from verbs.rb; requires net/http and the Hash#x_www_form_urlencode extension.
+2. + lib/HTTP/request_with_body.rb: HTTP.request_with_body, extracted from verbs.rb; requires net/http and json.
+3. ~ HTTP/verbs.rb: the request builders are extracted; the file now requires the two new files and keeps the VERBS constant and the named-method generation.
+4. ~ HTTP::VERSION: /1.2.0/1.2.1/
+5. ~ CHANGELOG: + 1.2.1 entry
+
+## 20260610
 
 1.2.0: Extract request_without_body and request_with_body.
 

data/README.md

@@ -110,10 +110,10 @@ HTTP.get('http://example.com', {}, {}, {retries: 3})
 
 When enabled, transient network exceptions and retry-worthy HTTP status codes (429, 502, 503, 504) are retried with exponential backoff and jitter. If the response carries a `Retry-After` header, it is honoured in place of the calculated delay.
 
-Only idempotent verbs (`get`, `head`, `options`, `put`, `delete`, `trace`) are retried by default. POST and PATCH are not — retrying a non-idempotent write can create duplicate resources against APIs that don't deduplicate. Opt in per-call via `retry_verbs:`.
+Only idempotent methods (`get`, `head`, `options`, `put`, `delete`, `trace`) are retried by default. POST and PATCH are not — retrying a non-idempotent write can create duplicate resources against APIs that don't deduplicate. Opt in per-call via `retry_methods:` (the older `retry_verbs:` is still accepted).
 
 ```ruby
-HTTP.post('http://example.com', {a: 1}, {}, {retries: 3, retry_verbs: %i{get post}})
+HTTP.post('http://example.com', {a: 1}, {}, {retries: 3, retry_methods: %i{get post}})
 ```
 
 Configurable options:
@@ -124,7 +124,7 @@ options = {
   retry_delay: 1.0,                             # base delay (seconds) for exponential backoff
   retry_status_codes: [429, 502, 503, 504],     # HTTP status codes to retry
   retry_exceptions: HTTP::RETRY::EXCEPTIONS,    # exceptions to retry
-  retry_verbs: HTTP::RETRY::VERBS               # verbs that retry by default
+  retry_methods: HTTP::RETRY::METHODS           # methods that retry by default (retry_verbs still accepted)
 }
 ```
 

data/lib/HTTP/METHODS.rb

@@ -0,0 +1,29 @@
+# HTTP/METHODS.rb
+# HTTP::METHODS
+
+require_relative './request_without_body'
+require_relative './request_with_body'
+
+module HTTP
+  module METHODS
+    WITHOUT_BODY = %i{get delete head options trace}
+    WITH_BODY = %i{post put patch}
+  end
+
+  VERBS = METHODS # Deprecated alias for METHODS; to be removed in 2.0.0.
+  deprecate_constant :VERBS
+
+  METHODS::WITHOUT_BODY.each do |request_method|
+    define_method(request_method) do |uri, args = {}, headers = {}, options = {}, &block|
+      request_without_body(request_method, uri, args, headers, options, &block)
+    end
+    module_function request_method
+  end
+
+  METHODS::WITH_BODY.each do |request_method|
+    define_method(request_method) do |uri, data = {}, headers = {}, options = {}, &block|
+      request_with_body(request_method, uri, data, headers, options, &block)
+    end
+    module_function request_method
+  end
+end

data/lib/HTTP/RETRY.rb

@@ -19,7 +19,9 @@ module HTTP
       EOFError
     ].freeze
     STATUS_CODES = [429, 502, 503, 504].freeze
-    VERBS = %i{get head options put delete trace}.freeze
+    METHODS = %i{get head options put delete trace}.freeze
+    VERBS = METHODS # Deprecated alias for METHODS; to be removed in 2.0.0.
+    deprecate_constant :VERBS
 
     def self.sleep(seconds)
       Kernel.sleep(seconds)
@@ -32,7 +34,7 @@ module HTTP
       delay: options.delete(:retry_delay) || 1.0,
       status_codes: options.delete(:retry_status_codes) || RETRY::STATUS_CODES,
       exceptions: options.delete(:retry_exceptions) || RETRY::EXCEPTIONS,
-      verbs: options.delete(:retry_verbs) || RETRY::VERBS
+      methods: [options.delete(:retry_methods), options.delete(:retry_verbs)].compact.first || RETRY::METHODS
     }
   end
   module_function :retry_config

data/lib/HTTP/VERSION.rb

@@ -2,5 +2,5 @@
 # HTTP::VERSION
 
 module HTTP
-  VERSION = '1.2.0'
+  VERSION = '1.3.0'
 end

data/lib/HTTP/request.rb

@@ -29,9 +29,9 @@ module HTTP
     elsif uri.user
       request_object.basic_auth(uri.user, uri.password)
     end
-    verb = request_object.method.downcase.to_sym
+    request_method = request_object.method.downcase.to_sym
     response = (
-      if config[:retries] > 0 && config[:verbs].include?(verb)
+      if config[:retries] > 0 && config[:methods].include?(request_method)
         with_retries(http, request_object, config)
       else
         http.request(request_object)
@@ -45,8 +45,8 @@ module HTTP
       end
       redirect_uri = uri.merge(response['location'])
       if response.code =~ /^30[78]$/
-        data = VERBS::WITH_BODY.include?(verb) ? request_object.body : {}
-        response = send(verb, redirect_uri.to_s, data, headers, options, &block)
+        data = METHODS::WITH_BODY.include?(request_method) ? request_object.body : {}
+        response = send(request_method, redirect_uri.to_s, data, headers, options, &block)
       else
         response = get(redirect_uri.to_s, {}, {}, options, &block)
       end

data/lib/HTTP/request_with_body.rb

@@ -0,0 +1,25 @@
+# HTTP/request_with_body.rb
+# HTTP.request_with_body
+
+require 'json'
+require 'net/http'
+
+require_relative './request'
+
+module HTTP
+  def request_with_body(method, uri, data = {}, headers = {}, options = {}, &block)
+    uri = uri.is_a?(URI) ? uri : URI.parse(uri)
+    request_object = Net::HTTP.const_get(method.to_s.capitalize).new(uri.request_uri)
+    content_type = headers.find{|k, v| k.downcase == 'content-type'}&.last.to_s
+    if data.is_a?(String)
+      request_object.body = data
+    elsif content_type.start_with?('application/json')
+      request_object.body = JSON.dump(data)
+    else
+      request_object.form_data = data
+    end
+    request(uri, request_object, headers, options, &block)
+  end
+
+  module_function :request_with_body
+end

data/lib/HTTP/request_without_body.rb

@@ -0,0 +1,21 @@
+# HTTP/request_without_body.rb
+# HTTP.request_without_body
+
+require 'net/http'
+
+require_relative '../Hash/x_www_form_urlencode'
+require_relative './request'
+
+module HTTP
+  def request_without_body(method, uri, args = {}, headers = {}, options = {}, &block)
+    uri = uri.is_a?(URI) ? uri : URI.parse(uri)
+    request_uri = uri.request_uri
+    unless args.empty?
+      request_uri += '?' + args.x_www_form_urlencode
+    end
+    request_object = Net::HTTP.const_get(method.to_s.capitalize).new(request_uri)
+    request(uri, request_object, headers, options, &block)
+  end
+
+  module_function :request_without_body
+end

data/lib/HTTP.rb

@@ -1,4 +1,4 @@
 # HTTP.rb
 # HTTP
 
-require 'HTTP/verbs'
+require 'HTTP/METHODS'

data/test/HTTP/METHODS_test.rb

@@ -0,0 +1,39 @@
+# test/HTTP/METHODS_test.rb
+
+require_relative '../helper'
+
+def without_deprecation_warnings
+  original = Warning[:deprecated]
+  Warning[:deprecated] = false
+  yield
+ensure
+  Warning[:deprecated] = original
+end
+
+describe "HTTP::METHODS" do
+  it "groups the methods sent without a body" do
+    _(HTTP::METHODS::WITHOUT_BODY).must_equal(%i{get delete head options trace})
+  end
+
+  it "groups the methods sent with a body" do
+    _(HTTP::METHODS::WITH_BODY).must_equal(%i{post put patch})
+  end
+
+  it "remains reachable under the deprecated VERBS name" do
+    without_deprecation_warnings do
+      _(HTTP::VERBS).must_equal(HTTP::METHODS)
+    end
+  end
+end
+
+describe "HTTP::RETRY::METHODS" do
+  it "lists the idempotent methods retried by default" do
+    _(HTTP::RETRY::METHODS).must_equal(%i{get head options put delete trace})
+  end
+
+  it "remains reachable under the deprecated VERBS name" do
+    without_deprecation_warnings do
+      _(HTTP::RETRY::VERBS).must_equal(HTTP::RETRY::METHODS)
+    end
+  end
+end

data/test/HTTP/RETRY_test.rb

@@ -177,6 +177,7 @@ describe "retry behaviour" do
           retry_delay: 0.1,
           retry_status_codes: [500],
           retry_exceptions: [Errno::ECONNRESET],
+          retry_methods: %i{get},
           retry_verbs: %i{get}
         })
       end
@@ -184,6 +185,7 @@ describe "retry behaviour" do
       _(received_opts).wont_include(:retry_delay)
       _(received_opts).wont_include(:retry_status_codes)
       _(received_opts).wont_include(:retry_exceptions)
+      _(received_opts).wont_include(:retry_methods)
       _(received_opts).wont_include(:retry_verbs)
     end
   end
@@ -240,6 +242,24 @@ describe "retry behaviour" do
         assert_requested(:post, uri, times: 2)
       end
     end
+
+    it "retries POST when opted in via retry_methods" do
+      HTTP::RETRY.stub(:sleep, nil) do
+        stub_request(:post, uri).to_return({status: 503}, {status: 200, body: ''})
+        response = HTTP.post(uri, {}, {}, {retries: 3, retry_methods: %i{get post}})
+        _(response.success?).must_equal(true)
+        assert_requested(:post, uri, times: 2)
+      end
+    end
+
+    it "prefers retry_methods over retry_verbs when both are given" do
+      HTTP::RETRY.stub(:sleep, nil) do
+        stub_request(:post, uri).to_return({status: 503}, {status: 200, body: ''})
+        response = HTTP.post(uri, {}, {}, {retries: 3, retry_methods: %i{get post}, retry_verbs: %i{get}})
+        _(response.success?).must_equal(true)
+        assert_requested(:post, uri, times: 2)
+      end
+    end
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http.rb
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - thoran
@@ -94,16 +94,19 @@ files:
 - Rakefile
 - http.rb.gemspec
 - lib/HTTP.rb
+- lib/HTTP/METHODS.rb
 - lib/HTTP/RETRY.rb
 - lib/HTTP/VERSION.rb
 - lib/HTTP/request.rb
-- lib/HTTP/verbs.rb
+- lib/HTTP/request_with_body.rb
+- lib/HTTP/request_without_body.rb
 - lib/Hash/x_www_form_urlencode.rb
 - lib/Net/HTTP/set_options.rb
 - lib/Net/HTTPRequest/set_headers.rb
 - lib/Net/HTTPResponse/StatusPredicates.rb
 - lib/String/url_encode.rb
 - lib/URI/Generic/use_sslQ.rb
+- test/HTTP/METHODS_test.rb
 - test/HTTP/RETRY_test.rb
 - test/HTTP/basic_auth_test.rb
 - test/HTTP/delete_test.rb

data/lib/HTTP/verbs.rb

@@ -1,55 +0,0 @@
-# HTTP/verbs.rb
-# HTTP.verb
-
-require 'json'
-require 'net/http'
-
-require_relative '../Hash/x_www_form_urlencode'
-require_relative './request'
-
-module HTTP
-  module VERBS
-    WITHOUT_BODY = %i{get delete head options trace}
-    WITH_BODY = %i{post put patch}
-  end
-
-  def request_without_body(method, uri, args = {}, headers = {}, options = {}, &block)
-    uri = uri.is_a?(URI) ? uri : URI.parse(uri)
-    request_uri = uri.request_uri
-    unless args.empty?
-      request_uri += '?' + args.x_www_form_urlencode
-    end
-    request_object = Net::HTTP.const_get(method.to_s.capitalize).new(request_uri)
-    request(uri, request_object, headers, options, &block)
-  end
-  module_function :request_without_body
-
-  def request_with_body(method, uri, data = {}, headers = {}, options = {}, &block)
-    uri = uri.is_a?(URI) ? uri : URI.parse(uri)
-    request_object = Net::HTTP.const_get(method.to_s.capitalize).new(uri.request_uri)
-    content_type = headers.find{|k, v| k.downcase == 'content-type'}&.last.to_s
-    if data.is_a?(String)
-      request_object.body = data
-    elsif content_type.start_with?('application/json')
-      request_object.body = JSON.dump(data)
-    else
-      request_object.form_data = data
-    end
-    request(uri, request_object, headers, options, &block)
-  end
-  module_function :request_with_body
-
-  VERBS::WITHOUT_BODY.each do |verb|
-    define_method(verb) do |uri, args = {}, headers = {}, options = {}, &block|
-      request_without_body(verb, uri, args, headers, options, &block)
-    end
-    module_function verb
-  end
-
-  VERBS::WITH_BODY.each do |verb|
-    define_method(verb) do |uri, data = {}, headers = {}, options = {}, &block|
-      request_with_body(verb, uri, data, headers, options, &block)
-    end
-    module_function verb
-  end
-end