Remove route_info from params (Close #789)

ruby-grape · Jan 5, 2015 · 2f07af6 · 2f07af6
1 parent 811eca6
commit 2f07af6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/grape/http/request.rb b/lib/grape/http/request.rb
@@ -7,6 +7,7 @@ def params
           args = env['rack.routing_args'].dup
           # preserve version from query string parameters
           args.delete(:version)
+          args.delete(:route_info)
           params.deep_merge!(args)
         end
         params

diff --git a/spec/grape/integration/rack_spec.rb b/spec/grape/integration/rack_spec.rb
@@ -23,7 +23,7 @@
       # can't check explicitly version of Rack because of https://github.com/rack/rack/issues/773
       pending 'Rack 1.6.0 is required' unless ::Rack.const_defined?(:TempfileReaper) || RUBY_PLATFORM == 'java'
 
-      expect(JSON.parse(app.call(env)[2].body.first)['params_keys']).to match_array(%w(route_info test))
+      expect(JSON.parse(app.call(env)[2].body.first)['params_keys']).to match_array('test')
     ensure
       input.close
       input.unlink