Page MenuHomePhabricator
Differential D19084

When "phabricator.silent" is enabled, don't call webhooks
ClosedPublic
Actions

Authored by epriestley on Feb 14 2018, 7:14 PM.
Tags
None
Referenced Files
F19922729: D19084.id45742.diff
Mon, Apr 6, 8:44 PM
F19855281: D19084.diff
Fri, Mar 13, 1:38 PM
F19851552: D19084.diff
Thu, Mar 12, 11:52 PM
F19810196: D19084.diff
Mar 3 2026, 6:07 PM
F19637317: D19084.diff
Feb 5 2026, 2:05 PM
F18877526: D19084.diff
Nov 6 2025, 9:46 AM
F18855314: D19084.id.diff
Nov 1 2025, 4:51 AM
F18780265: D19084.id45742.diff
Oct 12 2025, 4:43 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13078: `phabricator.silent` should also silence remote builds and webhooks
Commits
rPa2453706abf1: When "phabricator.silent" is enabled, don't call webhooks
Summary

Ref T13078. The phabricator.silent configuration flag should disable webhook calls, since this is consistent with the documented and desired behavior.

Test Plan

Enabled phabricator.silent, made test hook calls, saw them fail with a "silent" failure reason.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 14 2018, 7:14 PM
Harbormaster completed remote builds in B19546: Diff 45742.Feb 14 2018, 7:15 PM
epriestley requested review of this revision.Feb 14 2018, 7:15 PM
epriestley added a child revision: D19085: Make "phabricator.silent" disable build steps which rely on external services.Feb 14 2018, 7:36 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 14 2018, 7:40 PM
Closed by commit rPa2453706abf1: When "phabricator.silent" is enabled, don't call webhooks (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
worker/
7 lines

Diff 45744

View Options

src/applications/herald/worker/HeraldWebhookWorker.php

Loading...
Phabricator · Built by Phacility