Page MenuHomePhabricator
Differential D20924

Remove "phlog()" of exeptions during Conduit calls
ClosedPublic
Actions

Authored by epriestley on Nov 22 2019, 7:54 PM.
Tags
None
Referenced Files
F18734118: D20924.id.diff
Tue, Sep 30, 10:48 PM
F18694060: D20924.id.diff
Sat, Sep 27, 2:30 AM
F18646433: D20924.id.diff
Fri, Sep 19, 9:55 AM
F18584227: D20924.diff
Sep 11 2025, 12:22 PM
F18567570: D20924.id.diff
Sep 9 2025, 3:56 PM
F18141911: D20924.id49866.diff
Aug 14 2025, 10:47 AM
F18141821: D20924.id49865.diff
Aug 14 2025, 9:59 AM
F18112300: D20924.id49866.diff
Aug 12 2025, 4:19 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13465: Stop sending Conduit exceptions to the error log
Commits
rPeb6df7a2091a: Remove "phlog()" of exeptions during Conduit calls
Summary

Fixes T13465. This "phlog()" made some degree of sense at one time, but is no longer useful or consistent. Get rid of it. See T13465 for discussion.

Test Plan

Made a conduit call that hit a policy error, no longer saw error in log.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 22 2019, 7:54 PM
Harbormaster completed remote builds in B23694: Diff 49865.Nov 22 2019, 7:56 PM
epriestley requested review of this revision.Nov 22 2019, 7:56 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 22 2019, 8:03 PM
Closed by commit rPeb6df7a2091a: Remove "phlog()" of exeptions during Conduit calls (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPeb6df7a2091a: Remove "phlog()" of exeptions during Conduit calls.

Revision Contents
Changeset List

PathSize
src/
applications/
conduit/
controller/
3 lines

Diff 49866

View Options

src/applications/conduit/controller/PhabricatorConduitAPIController.php

Loading...
Phabricator · Built by Phacility