Page MenuHomePhabricator

Remove legacy transaction editor "getDisableEmail()" method
ClosedPublic

Authored by epriestley on Jan 19 2018, 3:48 PM.
Tags
None
Referenced Files
F13131616: D18881.id.diff
Wed, May 1, 5:15 PM
Unknown Object (File)
Tue, Apr 30, 7:58 AM
Unknown Object (File)
Thu, Apr 25, 12:59 AM
Unknown Object (File)
Sat, Apr 6, 12:18 AM
Unknown Object (File)
Fri, Apr 5, 7:37 PM
Unknown Object (File)
Mar 11 2024, 12:32 AM
Unknown Object (File)
Feb 26 2024, 10:13 AM
Unknown Object (File)
Feb 26 2024, 10:13 AM
Subscribers
None

Details

Summary

Ref T13042. This is a very, very old policy-violating option from yesteryear which supported build systems publishing updates by adding comments to revisions, without sending email about it.

Harbormaster has served this role for a long time and this is policy-violating in the general case (it allows attackers to act in secret).

Test Plan

Grepped for affected symbols.

Diff Detail

Repository
rP Phabricator
Branch
bulk16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19057
Build 25719: Run Core Tests
Build 25718: arc lint + arc unit