http.rb 0.18.1 → 0.18.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3cf46d2fcd2e4e18c1bd5fb1245da1156b14d694871970fb6d827d5a79f2a533
-  data.tar.gz: 786e1c2b4fc5314e2626f466bfd249d7a24ea80b607ed0f3c67333c6220147e2
+  metadata.gz: 5cdc1621048e0c428eb0b0c2ae81ae534921007746d58b426cc1185e6a77557a
+  data.tar.gz: 80f9b3f58b9c410093e74243b6a23ac364a50c3579c47ddf51e3e77e9b80a769
 SHA512:
-  metadata.gz: 86963812021231537e441627cbe6e617920409c465010296e4f513f838c40df51005edfc25083ad03348566de02c94a80ae9a3ccfa26c956abe00274425857aa
-  data.tar.gz: d5b6e217124fffa5ee8a23e78c316e9d8bbce7cfe8523613923ac18b34bfa74e003a21294b793ddca56785261ea8f765e158adae3b83663d732aa19c12c38fcc
+  metadata.gz: 63d22c4b9ed0a909a8a72dcf9cf00f44d0aad730f5e6bdd0aef6a70616bf27b12b81737ee71f5e5d368d46142355fe7046cafe633d5e6b3b744b4c19c8e42d16
+  data.tar.gz: f5bc846fae0969a4892926c1d739a549de54dc1a4765a4996bcbf56144c4d56bfedfb1044fd7399d3cfcfb7cfcd4ee4eca88c11c8c93433b665f1094d96ddd2f

data/CHANGELOG

@@ -1,5 +1,15 @@
 # CHANGELOG
 
+# 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.
+2. ~ spec/HTTP/get_spec.rb: Update relative-redirect stubs to elide default port; + context for HTTPS relative redirect.
+3. ~ spec/HTTP/post_spec.rb: Update relative-redirect stubs to elide default port.
+4. ~ spec/HTTP/put_spec.rb: Update relative-redirect stubs to elide default port.
+5. ~ spec/HTTP/delete_spec.rb: Update relative-redirect stubs to elide default port.
+6. ~ HTTP::VERSION: /0.18.1/0.18.2/
+7. ~ CHANGELOG: + 0.18.2 entry
+
 # 20260508
 # 0.18.1: Remove incorrectly added WebDAV verbs.
 1. - lib/Net/HTTP/Report.rb

data/lib/HTTP/VERSION.rb

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

data/lib/HTTP/request.rb

@@ -27,13 +27,8 @@ module HTTP
       elsif no_redirect
         return response
       end
-      redirect_uri = URI.parse(response.header['location'])
-      if redirect_uri.scheme
-        response = get(response.header['location'], {}, {}, options, &block)
-      else
-        new_location = "http://#{uri.host}:#{uri.port}#{response.header['location']}"
-        response = get(new_location, {}, {}, options, &block)
-      end
+      redirect_uri = uri.merge(response.header['location'])
+      response = get(redirect_uri.to_s, {}, {}, options, &block)
     end
     if block_given?
       yield response

data/spec/HTTP/delete_spec.rb

@@ -173,7 +173,7 @@ describe ".delete" do
   context "with path only redirection" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_path){'/new_path'}
-    let(:redirect_uri){"http://example.com:80#{redirect_path}"}
+    let(:redirect_uri){"http://example.com#{redirect_path}"}
 
     before do
       stub_request(:get, redirect_uri).

data/spec/HTTP/get_spec.rb

@@ -173,7 +173,7 @@ describe ".get" do
   context "with path only redirection" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_path){'/new_path'}
-    let(:redirect_uri){"http://example.com:80#{redirect_path}"}
+    let(:redirect_uri){"http://example.com#{redirect_path}"}
 
     before do
       stub_request(:get, redirect_uri).
@@ -212,6 +212,28 @@ describe ".get" do
     end
   end
 
+  context "with path only redirection from HTTPS" do
+    let(:request_uri){'https://example.com/path'}
+    let(:redirect_path){'/new_path'}
+    let(:redirect_uri){"https://example.com#{redirect_path}"}
+
+    before do
+      stub_request(:get, 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: {})
+      stub_request(:get, 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: 301, body: '', headers: {'location' => redirect_path})
+    end
+
+    it "preserves the HTTPS scheme on a relative redirect" do
+      expect(HTTP).to receive(:get).once.with(request_uri).and_call_original
+      expect(HTTP).to receive(:get).once.with(redirect_uri, {}, {}, {use_ssl: true, verify_mode: 0}).and_call_original
+      response = HTTP.get(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

@@ -311,7 +311,7 @@ describe ".post" do
   context "with path only redirection" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_path){'/new_path'}
-    let(:redirect_uri){"http://example.com:80#{redirect_path}"}
+    let(:redirect_uri){"http://example.com#{redirect_path}"}
 
     before do
       stub_request(:get, redirect_uri).

data/spec/HTTP/put_spec.rb

@@ -311,7 +311,7 @@ describe ".put" do
   context "with path only redirection" do
     let(:request_uri){'http://example.com/path'}
     let(:redirect_path){'/new_path'}
-    let(:redirect_uri){"http://example.com:80#{redirect_path}"}
+    let(:redirect_uri){"http://example.com#{redirect_path}"}
 
     before do
       stub_request(:get, redirect_uri).

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http.rb
 version: !ruby/object:Gem::Version
-  version: 0.18.1
+  version: 0.18.2
 platform: ruby
 authors:
 - thoran
@@ -79,6 +79,7 @@ files:
 - README.md
 - Rakefile
 - http.rb.gemspec
+- lib/HTTP.rb
 - lib/HTTP/VERSION.rb
 - lib/HTTP/request.rb
 - lib/HTTP/verbs.rb
@@ -92,7 +93,6 @@ files:
 - lib/Thoran/Array/FirstX/firstX.rb
 - lib/Thoran/String/ToConst/to_const.rb
 - lib/URI/Generic/use_sslQ.rb
-- lib/http.rb
 - spec/HTTP/delete_spec.rb
 - spec/HTTP/get_spec.rb
 - spec/HTTP/head_spec.rb
@@ -117,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.11
+rubygems_version: 4.0.12
 specification_version: 4
 summary: HTTP made easy.
 test_files: []