http.rb 0.18.2 → 0.18.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5cdc1621048e0c428eb0b0c2ae81ae534921007746d58b426cc1185e6a77557a
-  data.tar.gz: 80f9b3f58b9c410093e74243b6a23ac364a50c3579c47ddf51e3e77e9b80a769
+  metadata.gz: 4bfdcd1317baede9f978ecc2090b816da19fd76531ab759ff685ffc42e237297
+  data.tar.gz: a4ea5421285e8a5b1b702b33be58de8be2e8984963b238b81679fdbca118eeec
 SHA512:
-  metadata.gz: 63d22c4b9ed0a909a8a72dcf9cf00f44d0aad730f5e6bdd0aef6a70616bf27b12b81737ee71f5e5d368d46142355fe7046cafe633d5e6b3b744b4c19c8e42d16
-  data.tar.gz: f5bc846fae0969a4892926c1d739a549de54dc1a4765a4996bcbf56144c4d56bfedfb1044fd7399d3cfcfb7cfcd4ee4eca88c11c8c93433b665f1094d96ddd2f
+  metadata.gz: 5dbf7b38407e2bcc6932efdee8b36ee4994cdfc420edcb6d2c7f39d9a7c407af2c2b786424d0b33123e4538b7aa094e7ed0ea5597d1afbe392aad913b29e890c
+  data.tar.gz: ad271ead133a0f244d4f1538ee78aa2f6595c0a0c7be47bbf5d965f8768136f579354a1d4015754e0d60e922e64ade6cbc9043213d63d006c50e140e3c196553

data/CHANGELOG

@@ -1,5 +1,15 @@
 # CHANGELOG
 
+# 20260521
+# 0.18.3: Fix verb preservation on 307/308 redirects.
+1. ~ HTTP.request: Use original verb when following 307 or 308 redirects, per RFC 7231 §6.4.7 and RFC 7538. 301/302/303 keep legacy GET-on-redirect behaviour.
+2. ~ spec/HTTP/post_spec.rb: + specs for 307/308 verb preservation and 307 body preservation.
+3. ~ spec/HTTP/put_spec.rb: + spec for 307 verb preservation.
+4. ~ spec/HTTP/delete_spec.rb: + spec for 307 verb preservation.
+5. ~ spec/spec_helper.rb: Skip webmock's http_rb adapter, which collides with this gem's HTTP module on case-insensitive filesystems.
+6. ~ HTTP::VERSION: /0.18.2/0.18.3/
+7. ~ CHANGELOG: + 0.18.3 entry
+
 # 20260520
 # 0.18.2: Fix relative redirect URL construction.
 1. ~ HTTP.request: Use URI#merge for redirect URL construction. Preserves original scheme, elides default ports, and resolves relative paths per RFC 3986.

data/lib/HTTP/VERSION.rb

@@ -2,5 +2,5 @@
 # HTTP::VERSION
 
 module HTTP
-  VERSION = '0.18.2'
+  VERSION = '0.18.3'
 end

data/lib/HTTP/request.rb

@@ -28,7 +28,13 @@ module HTTP
         return response
       end
       redirect_uri = uri.merge(response.header['location'])
-      response = get(redirect_uri.to_s, {}, {}, options, &block)
+      if response.code =~ /^30[78]$/
+        verb = request_object.method.downcase.to_sym
+        data = VERBS::WITH_BODY.include?(verb) ? request_object.body : {}
+        response = send(verb, redirect_uri.to_s, data, headers, options, &block)
+      else
+        response = get(redirect_uri.to_s, {}, {}, options, &block)
+      end
     end
     if block_given?
       yield response

data/spec/HTTP/delete_spec.rb

@@ -212,6 +212,27 @@ describe ".delete" do
     end
   end
 
+  context "with verb-preserving redirection via 307" do
+    let(:request_uri){'http://example.com/path'}
+    let(:redirect_uri){'http://redirected.com'}
+
+    before do
+      stub_request(:delete, request_uri).
+        with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+          to_return(status: 307, body: '', headers: {'location' => redirect_uri})
+      stub_request(:delete, redirect_uri).
+        with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+          to_return(status: 200, body: '', headers: {})
+    end
+
+    it "preserves the verb" do
+      expect(HTTP).to receive(:delete).with(request_uri).and_call_original.ordered
+      expect(HTTP).to receive(:delete).with(redirect_uri, {}, {}, {use_ssl: false, verify_mode: 0}).and_call_original.ordered
+      response = HTTP.delete(request_uri)
+      expect(response.success?).to eq(true)
+    end
+  end
+
   context "no_redirect true" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_uri){'http://redirected.com'}

data/spec/HTTP/post_spec.rb

@@ -350,6 +350,68 @@ describe ".post" do
     end
   end
 
+  context "with verb-preserving redirection" do
+    let(:request_uri){'http://example.com/path'}
+    let(:redirect_uri){'http://redirected.com'}
+
+    before do
+      stub_request(:post, redirect_uri).
+        with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+          to_return(status: 200, body: '', headers: {})
+    end
+
+    context "via 307" do
+      before do
+        stub_request(:post, request_uri).
+          with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+            to_return(status: 307, body: '', headers: {'location' => redirect_uri})
+      end
+
+      it "preserves the verb" do
+        expect(HTTP).to receive(:post).with(request_uri).and_call_original.ordered
+        expect(HTTP).to receive(:post).with(redirect_uri, '', {}, {use_ssl: false, verify_mode: 0}).and_call_original.ordered
+        response = HTTP.post(request_uri)
+        expect(response.success?).to eq(true)
+      end
+    end
+
+    context "via 308" do
+      before do
+        stub_request(:post, request_uri).
+          with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+            to_return(status: 308, body: '', headers: {'location' => redirect_uri})
+      end
+
+      it "preserves the verb" do
+        expect(HTTP).to receive(:post).with(request_uri).and_call_original.ordered
+        expect(HTTP).to receive(:post).with(redirect_uri, '', {}, {use_ssl: false, verify_mode: 0}).and_call_original.ordered
+        response = HTTP.post(request_uri)
+        expect(response.success?).to eq(true)
+      end
+    end
+  end
+
+  context "with body-preserving redirection via 307" do
+    let(:request_uri){'http://example.com/path'}
+    let(:redirect_uri){'http://redirected.com'}
+    let(:args) do; {a: 1, b: 2}; end
+    let(:encoded_form_data){args.x_www_form_urlencode}
+
+    before do
+      stub_request(:post, request_uri).
+        with(body: encoded_form_data).
+          to_return(status: 307, body: '', headers: {'location' => redirect_uri})
+      stub_request(:post, redirect_uri).
+        with(body: encoded_form_data).
+          to_return(status: 200, body: '', headers: {})
+    end
+
+    it "preserves the body" do
+      response = HTTP.post(request_uri, args)
+      expect(response.success?).to eq(true)
+    end
+  end
+
   context "no_redirect true" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_uri){'http://redirected.com'}

data/spec/HTTP/put_spec.rb

@@ -350,6 +350,27 @@ describe ".put" do
     end
   end
 
+  context "with verb-preserving redirection via 307" do
+    let(:request_uri){'http://example.com/path'}
+    let(:redirect_uri){'http://redirected.com'}
+
+    before do
+      stub_request(:put, request_uri).
+        with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+          to_return(status: 307, body: '', headers: {'location' => redirect_uri})
+      stub_request(:put, redirect_uri).
+        with(headers: {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Ruby'}).
+          to_return(status: 200, body: '', headers: {})
+    end
+
+    it "preserves the verb" do
+      expect(HTTP).to receive(:put).with(request_uri).and_call_original.ordered
+      expect(HTTP).to receive(:put).with(redirect_uri, '', {}, {use_ssl: false, verify_mode: 0}).and_call_original.ordered
+      response = HTTP.put(request_uri)
+      expect(response.success?).to eq(true)
+    end
+  end
+
   context "no_redirect true" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_uri){'http://redirected.com'}

data/spec/spec_helper.rb

@@ -3,4 +3,13 @@
 lib_dir = File.expand_path(File.join(__FILE__, '..', '..', 'lib'))
 $LOAD_PATH.unshift(lib_dir) unless $LOAD_PATH.include?(lib_dir)
 
+# Skip webmock's http_rb adapter: it auto-requires 'http', which collides
+# with this gem's lib/HTTP.rb on case-insensitive filesystems and crashes
+# while trying to monkey-patch the wrong HTTP module.
+gem 'webmock'
+$LOADED_FEATURES << File.join(
+  Gem.loaded_specs['webmock'].full_gem_path,
+  'lib/webmock/http_lib_adapters/http_rb_adapter.rb'
+)
+
 require 'webmock/rspec'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http.rb
 version: !ruby/object:Gem::Version
-  version: 0.18.2
+  version: 0.18.3
 platform: ruby
 authors:
 - thoran