quickfix for jruby + rack-reverse-proxy + rspec-mocks + :read_timeout= weirdness

Author: waterlink

lib/rack/reverse_proxy.rb

@@ -3,6 +3,7 @@
 require "rack-proxy"
 require "rack/reverse_proxy_matcher"
 require "rack/exception"
+require "rack/reverse_proxy/http_streaming_response"
 
 module Rack
   class ReverseProxy
@@ -72,7 +73,7 @@ def proxy(env, source_request, matcher)
       target_response = HttpStreamingResponse.new(target_request, uri.host, uri.port)
 
       # pass the timeout configuration through
-      target_response.read_timeout = options[:timeout] if options[:timeout].to_i > 0
+      target_response.set_read_timeout(options[:timeout]) if options[:timeout].to_i > 0
 
       target_response.use_ssl = "https" == uri.scheme
 

lib/rack/reverse_proxy/http_streaming_response.rb

@@ -0,0 +1,7 @@
+module Rack
+  class HttpStreamingResponse
+    def set_read_timeout(value)
+      self.read_timeout = value
+    end
+  end
+end

spec/rack/reverse_proxy_spec.rb

@@ -89,7 +89,7 @@ def app
 
       it "should make request with basic auth" do
         stub_request(:get, "http://example.com/test/slow")
-        Rack::HttpStreamingResponse.any_instance.should_receive(:read_timeout=).with(99)
+        Rack::HttpStreamingResponse.any_instance.should_receive(:set_read_timeout).with(99)
         get '/test/slow'
       end
     end
@@ -103,7 +103,7 @@ def app
 
       it "should make request with basic auth" do
         stub_request(:get, "http://example.com/test/slow")
-        Rack::HttpStreamingResponse.any_instance.should_not_receive(:read_timeout=)
+        Rack::HttpStreamingResponse.any_instance.should_not_receive(:set_read_timeout)
         get '/test/slow'
       end
     end