Diffusion Phabricator eb6df7a2091a

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

Description

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.

Maniphest Tasks: T13465

Differential Revision: https://secure.phabricator.com/D20924

Details

Provenance

epriestley	Authored on Nov 22 2019, 7:53 PM
epriestley	Pushed on Nov 22 2019, 8:03 PM

Differential Revision

D20924: Remove "phlog()" of exeptions during Conduit calls

Parents

rP2abf292821a2: Fix an issue where editing paths in Owners packages could raise an error…

Branches

Unknown

Tags

Unknown

Tasks

T13465: Stop sending Conduit exceptions to the error log

Build Status

Buildable 23695
Build 32577: Run Core Tests

Event Timeline

epriestley committed rPeb6df7a2091a: Remove "phlog()" of exeptions during Conduit calls (authored by epriestley).Nov 22 2019, 8:03 PM

epriestley added a task: T13465: Stop sending Conduit exceptions to the error log.

epriestley added an edge: D20924: Remove "phlog()" of exeptions during Conduit calls.

Harbormaster completed building B23695: rPeb6df7a2091a: Remove "phlog()" of exeptions during Conduit calls.Nov 22 2019, 8:04 PM

epriestley mentioned this in 2019 Week 47 (Very Late November).Nov 25 2019, 3:11 PM

Changes (1)

Path

Size

src/

applications/

conduit/

controller/

PhabricatorConduitAPIController.php

rPeb6df7a2091a

View Options

src/applications/conduit/controller/PhabricatorConduitAPIController.php