Page MenuHomePhabricator

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

Authored by epriestley on Feb 14 2018, 7:14 PM.
Tags
None
Referenced Files
F15456670: D19084.id45744.diff
Sun, Mar 30, 11:15 AM
F15455970: D19084.id45742.diff
Sun, Mar 30, 6:05 AM
F15454344: D19084.id.diff
Sat, Mar 29, 6:05 PM
F15449623: D19084.diff
Fri, Mar 28, 11:19 AM
F15417942: D19084.id45744.diff
Thu, Mar 20, 7:38 PM
F15393670: D19084.id45744.diff
Sat, Mar 15, 10:59 PM
F15393669: D19084.id45742.diff
Sat, Mar 15, 10:59 PM
F15391000: D19084.diff
Sat, Mar 15, 7:47 AM
Subscribers
None

Details

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

This revision was not accepted when it landed; it landed in state Needs Review.Feb 14 2018, 7:40 PM
This revision was automatically updated to reflect the committed changes.