allow PATCH verb to be processed by the ruby application

Sponsored by: Lookout, Inc.

Conflicts:
	src/main/java/org/jruby/rack/UnmappedRackFilter.java
	src/spec/ruby/rack/filter_spec.rb

Author: mkristian

src/main/java/org/jruby/rack/UnmappedRackFilter.java

@@ -48,8 +48,8 @@ public class UnmappedRackFilter extends AbstractFilter {
     private Collection<Integer> responseNotHandledStatuses =
         // 403 due containers not supporting PUT/DELETE correctly (Tomcat 6)
         // 405 returned by Jetty 7/8 on PUT/DELETE requests by default
-        Collections.unmodifiableList( Arrays.asList(404, 403, 405) );
-
+        // 501 is returned for non standard http verbs like PATCH
+        Collections.unmodifiableList( Arrays.asList(404, 403, 405, 501) );
     private RackContext context;
     private RackDispatcher dispatcher;
 

src/spec/ruby/rack/filter_spec.rb

@@ -85,6 +85,17 @@ def stub_request(path_info)
     dispatcher.should_receive(:process).ordered
     filter.doFilter(@request, @response, chain)
   end
+  
+  it "dispatches to the rack dispatcher if the chain resulted in a 501" do
+    # non standard verbs like PATCH produce HTTP 501
+    # see also http://httpstatus.es/501 and http://tools.ietf.org/html/rfc5789
+    chain.should_receive(:doFilter).ordered.and_return do |_, resp|
+      resp.sendError(501)
+    end
+    @response.should_receive(:reset)
+    dispatcher.should_receive(:process)
+    filter.doFilter(@request, @response, chain)
+  end
 
   it "dispatches to the rack dispatcher out of configured non handled statuses" do
     filter = Class.new(org.jruby.rack.RackFilter) do