Store original HTTP method in MethodOverride middleware

leahneukirchen · Jan 6, 2009 · e908d26 · e908d26
1 parent f981f3b
commit e908d26
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/rack/methodoverride.rb b/lib/rack/methodoverride.rb
@@ -16,6 +16,7 @@ def call(env)
           env[HTTP_METHOD_OVERRIDE_HEADER]
         method = method.to_s.upcase
         if HTTP_METHODS.include?(method)
+          env["rack.methodoverride.original_method"] = env["REQUEST_METHOD"]
           env["REQUEST_METHOD"] = method
         end
       end

diff --git a/test/spec_rack_methodoverride.rb b/test/spec_rack_methodoverride.rb
@@ -47,4 +47,14 @@
 
     req.env["REQUEST_METHOD"].should.equal "POST"
   end
+
+  specify "should store the original REQUEST_METHOD prior to overriding" do
+    env = Rack::MockRequest.env_for("/",
+            :method => "POST",
+            :input  => "_method=options")
+    app = Rack::MethodOverride.new(lambda { |env| Rack::Request.new(env) })
+    req = app.call(env)
+
+    req.env["rack.methodoverride.original_method"].should.equal "POST"
+  end
 end