Page MenuHomePhabricator

Allow Herald "diff" rules to reject content before it is written
ClosedPublic

Authored by epriestley on Aug 20 2014, 2:16 AM.
Tags
None
Referenced Files
F15498805: D10305.id24824.diff
Sun, Apr 13, 2:22 PM
F15478932: D10305.id24824.diff
Tue, Apr 8, 4:08 AM
F15475682: D10305.id24813.diff
Sun, Apr 6, 9:33 PM
F15468708: D10305.id.diff
Fri, Apr 4, 4:49 AM
F15467962: D10305.diff
Thu, Apr 3, 6:50 PM
F15418825: D10305.id24824.diff
Fri, Mar 21, 1:42 AM
F15414600: D10305.id24813.diff
Mar 20 2025, 1:26 AM
F15408840: D10305.id24824.diff
Mar 19 2025, 1:50 AM
Subscribers

Details

Summary

Fixes T5915. Occasionally, users derp up and diff private key material. Adding a pre-write Herald phase enables configuration of a partial layer of protection that will reject these changes before they hit disk, provided they can be detected by, e.g., filename.

Test Plan
  • Added a rule with checks on every field, verified they looked fine in the transcript.
  • Created some revisions to test those changes (I have a bunch of revision rules locally).
  • Verified rejects don't write transcripts to the database.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Allow Herald "diff" rules to reject content before it is written.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 20 2014, 8:16 PM
epriestley updated this revision to Diff 24824.

Closed by commit rP5ac36e8f77ee (authored by @epriestley).