Don't log Conduit 404s as errors
bff217efd314
Actions

Authored by Joshua Spence <josh@joshuaspence.com> on Jul 25 2014, 03:24.

Description

Don't log Conduit 404s as errors

Summary: Fixes T5695. A Conduit "method does not exist" exception is somewhat expected... there is no need to phlog the exception.

Test Plan: Called a non-existent Conduit method. Saw no exceptions in the error logs.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T5695

Committed

Joshua Spence <josh@joshuaspence.com>

Jul 25 2014, 03:24

Pushed

aubort

Jan 31 2017, 17:16

Parents

Branches

Unknown

Tags

Unknown

Joshua Spence <josh@joshuaspence.com> committed rPHbff217efd314: Don't log Conduit 404s as errors (authored by Joshua Spence <josh@joshuaspence.com>).Jul 25 2014, 03:24

		Path
M		src/__phutil_library_map__.php
M		src/applications/conduit/call/ConduitCall.php
M		src/applications/conduit/controller/PhabricatorConduitAPIController.php
V		src/applications/conduit/protocol/{ → exception}/ConduitException.php
A	(dir)	src/applications/conduit/protocol/exception/
A		src/applications/conduit/protocol/exception/ConduitApplicationNotInstalledException.php
V		src/applications/conduit/protocol/{exception ← }/ConduitException.php
A		src/applications/conduit/protocol/exception/ConduitMethodNotFoundException.php