Page MenuHomePhabricator

Don't log Conduit "method does not exist" exceptions
Closed, ResolvedPublic

Description

I'm seeing the following error in our logs. I think this is occuring when I run arc land (which usually queries Harbormaster for builds, except that we have Harbormaster uninstalled).

[23-Jul-2014 23:33:37 UTC] [2014-07-24 09:33:37] EXCEPTION: (ConduitException) Method 'ConduitAPI_harbormaster_querybuildables_Method' belongs to application 'Harbormaster', which is not installed. at [<phabricator>/src/application
s/conduit/call/ConduitCall.php:204]
[23-Jul-2014 23:33:37 UTC]   #0 ConduitCall::buildMethodHandler(string) called at [<phabricator>/src/applications/conduit/call/ConduitCall.php:21]
[23-Jul-2014 23:33:37 UTC]   #1 ConduitCall::__construct(string, array, boolean) called at [<phabricator>/src/applications/conduit/controller/PhabricatorConduitAPIController.php:36]
[23-Jul-2014 23:33:37 UTC]   #2 PhabricatorConduitAPIController::processRequest() called at [<phabricator>/webroot/index.php:95]

Revisions and Commits

Event Timeline

joshuaspence assigned this task to epriestley.
joshuaspence raised the priority of this task from to Needs Triage.
joshuaspence updated the task description. (Show Details)
joshuaspence added subscribers: joshuaspence, hach-que.

We should probably just stop logging this, I think the behavior here is otherwise reasonable/correct (i.e., have arc test for availability by just making the call). Roughly, this is like logging 404s as errors.

joshuaspence renamed this task from Harbormaster API call during `arc land` to Don't log Conduit "method does not exist" exceptions.Jul 24 2014, 10:34 PM
joshuaspence edited projects, added Conduit; removed Arcanist, Harbormaster.