do not include accept encoding by default + added ability to preserve encoding

Author: maxilev

lib/rack_reverse_proxy/roundtrip.rb

@@ -84,6 +84,11 @@ def preserve_host
       target_request_headers["HOST"] = host_header
     end
 
+    def preserve_encoding
+      return if options[:preserve_encoding]
+      target_request_headers.delete("Accept-Encoding")
+    end
+
     def host_header
       return uri.host if uri.port == uri.default_port
       "#{uri.host}:#{uri.port}"
@@ -178,6 +183,7 @@ def need_replace_location?
 
     def setup_request
       preserve_host
+      preserve_encoding
       set_forwarded_headers
       initialize_http_header
       set_basic_auth

spec/rack/reverse_proxy_spec.rb

@@ -147,6 +147,20 @@ def app
       expect(last_response.headers["Content-Length"]).to eq(body.length.to_s)
     end
 
+    it "does not include Accept-Encoding header" do
+      stub_request(:any, "http://example.com/test")
+
+      get "/test", {}, "HTTP_ACCEPT_ENCODING" => "gzip, deflate"
+
+      expect(
+        a_request(:get, "http://example.com/test").with(
+          :headers => { "Accept-Encoding" => "gzip, deflate" }
+        )
+      ).not_to have_been_made
+
+      expect(a_request(:get, "http://example.com/test")).to have_been_made
+    end
+
     describe "with non-default port" do
       def app
         Rack::ReverseProxy.new(dummy_app) do
@@ -186,6 +200,26 @@ def app
       end
     end
 
+    describe "with preserve encoding turned on" do
+      def app
+        Rack::ReverseProxy.new(dummy_app) do
+          reverse_proxy "/test", "http://example.com/", :preserve_encoding => true
+        end
+      end
+
+      it "sets the Accept-Encoding header" do
+        stub_request(:any, "http://example.com/test")
+
+        get "/test", {}, "HTTP_ACCEPT_ENCODING" => "gzip, deflate"
+
+        expect(
+          a_request(:get, "http://example.com/test").with(
+            :headers => { "Accept-Encoding" => "gzip, deflate" }
+          )
+        ).to have_been_made
+      end
+    end
+
     describe "with x_forwarded_headers turned off" do
       def app
         Rack::ReverseProxy.new(dummy_app) do