Неверно понимаю назначение param.require?

Question

[Everything_is_not_so_bad]

В контроллере нужно убедиться что параметры присутствуют. Если нет, то в API обработчик ActionController::ParameterMissing, как я понимаю, должен самостоятельно вернуть ответ об отсутствующих параметрах.

Тестирую так:

ActionController::Parameters.new(foo1: 'bar1', foo2: 'bar2').require([:foo1, :foo2])

ActionController::ParameterMissing: param is missing or the value is empty: [:foo1, :foo2]

Что нелогично, поскольку параметры то переданы. Что я делаю не так?

Ищу лаконичную альтернативу простым проверкам в контроллере типа таких:

(render json: {status: "Error", message: "Missing parameter 'foo1'"}; return) unless param.has_key?(:foo1)
(render json: {status: "Error", message: "Missing parameter 'foo2'"}; return) unless param.has_key?(:foo2)

Answer 1

[Михаил Белошицкий]

require не поддерживает массивы или несколько аргументов. Вызывайте по одиночке .

params = ActionController::Parameters.new(foo1: 'bar1', foo2: 'bar2')
params.require(:foo1)
params.require(:foo2)

Или воспользуйтесь map.

Comments

[Everything_is_not_so_bad]

Целью params.require являлась валидация аргументов. Как это правильно тогда сделать?

[Михаил Белошицкий]

Цель params.require - убедиться, что по указанные аргументы присутствуют. После этого еще обычно пишут .permit - это для того, чтобы отфильтровать явно не указанные ключи (вдруг там какая-нибудь бухня или злоумышленник пытается пропихнуть в параметрах какой-нибудь там :id). Валидацией занимаются валидаторы - guides.rubyonrails.org/active_record_validations.html

[Everything_is_not_so_bad]

Михаил Белошицкий: Михаил Белошицкий: Нет, я не имел ввиду валидацию аргументов модели. API не работает напрямую с моделями. Нужна именно проверка существования параметров в запросе.
Уточнил вопрос с примерами кода.

[Михаил Белошицкий]

В не зависимости от ваших уточнений "делаете не так" вы одно и то же - неправильно вызываете метод require.

Answer 2

[Everything_is_not_so_bad]

С подсказкой от Михаил Белошицкий пришёл к такой валидации аргументов:

params = ActionController::Parameters.new(foo1: 'bar1', invalid_foo: 'bar2') # только для демонстрации работы в консоли, в контроллере не строчка не нужна.
[:foo1, :foo2].each {|q| params.require(q)}

Answer 3

[Владислав Копылов]

Может вы перепутали require и permit?
apidock.com/rails/ActionController/Parameters/require
apidock.com/rails/ActionController/Parameters/permit

Comments

[Everything_is_not_so_bad]

У меня параметры используются для запроса GET, а не POST. Нет смысла заносить их в белый список при помощи permit, ибо нет никакого mass assignment. Зато нужно убедиться в наличии самих параметров в запросе.