Reject non-declared params

[dhulihan]

It would be nice if grape could reject a non-declared param. Something along the lines of:

params do
  requires :first_name, type: String

  reject_non_declared
end

If we specify age, it is rejected since because it has not been declared.

curl -d '{"first_name": "dave", "age": 32}' \
       'http://localhost:9292/statuses' \
       -H Content-Type:application/json \
       -v 
# => HTTP 400 

[dhulihan]

Here's my hacky solution for the time being:

module Helpers
  def extra_params
    @extra_params ||= begin
      declared_params = declared(params, include_missing: true).to_h
      params.to_h.reject { |k, v| declared_params.key?(k) }
    end
  end

  def reject_extra_params
    unless extra_params.empty?
      msg = "Unrecognized parameter(s) provided: #{extra_params}"
      raise Grape::Exceptions::Base.new(message: msg, status: 400)
    end
  end
end

class Foo < Grape::Api
  helpers Helpers

  post do
    reject_extra_params
    params
  end
end

[dblock]

This is a dup, sSee issues #810, #1603, and previous PR #1158. There's a new PR being attempted for this in #1606, you should check it out, seems like a cleaner solution than what you propose here.

[dhulihan]

K, thx. sry about the dup.

[dblock]

No worries @dhulihan!