Allow matchers to return nil and continue onto the original rack app

Author: markbates

lib/rack/reverse_proxy.rb

@@ -34,6 +34,9 @@ def call(env)
 
     def proxy(env, source_request, matcher)
       uri = matcher.get_uri(source_request.fullpath,env)
+      if uri.nil?
+        return @app.call(env)
+      end
       options = @global_options.dup.merge(matcher.options)
 
       # Initialize request

lib/rack/reverse_proxy_matcher.rb

@@ -21,6 +21,7 @@ def match?(path, *args)
     end
 
     def get_uri(path,env)
+      return nil if url.nil?
       _url=(url.respond_to?(:call) ? url.call(env) : url.clone)
       if _url =~/\$\d/
         match_path(path).to_a.each_with_index { |m, i| _url.gsub!("$#{i.to_s}", m) }

spec/rack/reverse_proxy_spec.rb

@@ -279,8 +279,6 @@ def self.match(path, headers, rackreq)
         def url(path)
           if rackreq.params["user"] == 'omer'
             'http://users-example.com' + path
-          else
-            'http://example.com' + path
           end
         end
       end
@@ -298,10 +296,9 @@ def app
       end
 
       it "should proxy requests when a pattern is matched" do
-        stub_request(:get, 'http://example.com/test?user=mark').to_return({:body => "Proxied App"})
         stub_request(:get, 'http://users-example.com/users?user=omer').to_return({:body => "User App"})
         get '/test', user: "mark"
-        last_response.body.should == "Proxied App"
+        last_response.body.should == "Dummy App"
         get '/users', user: 'omer'
         last_response.body.should == "User App"
       end