Page MenuHomePhabricator

When an invalid "constraints" parameter is provided to a "*.search" method, raise a more tailored error
ClosedPublic

Authored by epriestley on Mon, Apr 15, 6:50 PM.

Details

Summary

See https://discourse.phabricator-community.org/t/constraints-parameter-type-handling-in-maniphest-search/2636.

(The caller provided constraints={...} via cURL, but you can't mix JSON and HTTP parameters like that.)

Test Plan

Called curl 'http://local.phacility.com/api/maniphest.search?api.token=...&constraints=X' (with a valid API token), got a more sensible error.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Mon, Apr 15, 6:50 PM
epriestley requested review of this revision.Mon, Apr 15, 6:51 PM
amckinley accepted this revision.Wed, Apr 17, 8:10 PM
This revision is now accepted and ready to land.Wed, Apr 17, 8:10 PM
This revision was automatically updated to reflect the committed changes.