Merge pull request #410 from janko-m/keep-port-in-host-when-redirecting

Let Host be automatically inferred on redirects

lib/http/request.rb

@@ -94,17 +94,17 @@ def initialize(opts)
 
     # Returns new Request with updated uri
     def redirect(uri, verb = @verb)
-      req = self.class.new(
+      headers = self.headers.dup
+      headers.delete(Headers::HOST)
+
+      self.class.new(
         :verb    => verb,
         :uri     => @uri.join(uri),
         :headers => headers,
         :proxy   => proxy,
         :body    => body,
         :version => version
       )
-
-      req[Headers::HOST] = req.uri.host
-      req
     end
 
     # Stream the request to a socket

spec/lib/http/request_spec.rb

@@ -94,6 +94,20 @@
       expect(redirected["Host"]).to eq "blog.example.com"
     end
 
+    context "with URL with non-standard port given" do
+      subject(:redirected) { request.redirect "http://example.com:8080" }
+
+      its(:uri)     { is_expected.to eq HTTP::URI.parse "http://example.com:8080" }
+
+      its(:verb)    { is_expected.to eq request.verb }
+      its(:body)    { is_expected.to eq request.body }
+      its(:proxy)   { is_expected.to eq request.proxy }
+
+      it "presets new Host header" do
+        expect(redirected["Host"]).to eq "example.com:8080"
+      end
+    end
+
     context "with schema-less absolute URL given" do
       subject(:redirected) { request.redirect "//another.example.com/blog" }