adding global option matching => :first to request the first matched url

Author: Peter Schrammel

lib/rack/reverse_proxy.rb

@@ -6,7 +6,7 @@ class ReverseProxy
     def initialize(app = nil, &b)
       @app = app || lambda { [404, [], []] }
       @matchers = []
-      @global_options = {:preserve_host => false}
+      @global_options = {:preserve_host => false,:matching => :all}
       instance_eval &b if block_given?
     end
 
@@ -63,7 +63,7 @@ def get_matcher path
 
       if matches.length < 1
         nil
-      elsif matches.length > 1
+      elsif matches.length > 1 && @global_options[:matching] != :first
         raise AmbiguousProxyMatch.new(path, matches)
       else
         matches.first

spec/rack/reverse_proxy_spec.rb

@@ -77,9 +77,10 @@ def app
       end
     end
 
-    describe "with ambiguous routes" do
+    describe "with ambiguous routes and all matching" do
       def app
         Rack::ReverseProxy.new(dummy_app) do
+          reverse_proxy_options :matching => :all
           reverse_proxy '/test', 'http://example.com/'
           reverse_proxy /^\/test/, 'http://example.com/'
         end
@@ -90,6 +91,22 @@ def app
       end
     end
 
+    describe "with ambiguous routes and first matching" do
+      def app
+        Rack::ReverseProxy.new(dummy_app) do
+          reverse_proxy_options :matching => :first
+          reverse_proxy '/test', 'http://example1.com/'
+          reverse_proxy /^\/test/, 'http://example2.com/'
+        end
+      end
+
+      it "should throw an exception" do
+       stub_request(:get, 'http://example1.com/test').to_return({:body => "Proxied App"})
+       get '/test'
+       last_response.body.should == "Proxied App"
+      end
+    end
+
     describe "with a route as a regular expression" do
       def app
         Rack::ReverseProxy.new(dummy_app) do