Page MenuHomePhabricator

Add a Herald action to trigger "Must Encrypt" for mail
ClosedPublic

Authored by epriestley on Feb 1 2018, 5:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 23, 5:00 AM
Unknown Object (File)
Sun, Mar 10, 4:42 AM
Unknown Object (File)
Feb 11 2024, 11:32 PM
Unknown Object (File)
Dec 27 2023, 12:41 PM
Unknown Object (File)
Dec 23 2023, 11:10 AM
Unknown Object (File)
Dec 21 2023, 11:11 PM
Unknown Object (File)
Dec 21 2023, 10:16 AM
Unknown Object (File)
Dec 19 2023, 4:51 AM
Subscribers
None

Details

Summary

Depends on D18983. Ref T13053. Adds a new Herald action to activate the "must encrypt" flag and drop mail content.

Test Plan
  • Created a new Herald rule:

Screen Shot 2018-02-01 at 9.29.40 AM.png (1×1 px, 164 KB)

  • Created a "dog task" (woof woof, unsecure) and a "duck task" (quack quack, secure).
  • Viewed mail for both in bin/mail and web UI, saw appropriate security/encryption behavior.
  • Viewed "Must Encrypt" in "Headers" tab for the duck mail, saw why the mail was encrypted (link to Herald rule).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/metamta/herald/PhabricatorMailMustEncryptHeraldAction.php
20

Leftover debugging code?

59

"can be transmitted"?

This revision now requires changes to proceed.Feb 2 2018, 1:26 AM
src/applications/metamta/herald/PhabricatorMailMustEncryptHeraldAction.php
20

Yikes, thanks. Maybe I can write a lint rule to catch this.

59

I think there's technically a grammatical parsing of this, but upon re-reading it I agree it's pretty clumsy:

[ Made it required that ] mail content be transmitted only over secure channels.

In the UI, this phrase appears in the past tense (when looking at a transcript, explaining what actions Herald took) while the other one appears in the present tense (when looking at a rule, explaining what will happen when it matches) which is why the phrasing is a little weird.

  • Remove debugging code.
  • Rephrase to "Made it a requirement that mail content...".
This revision is now accepted and ready to land.Feb 2 2018, 10:11 PM
This revision was automatically updated to reflect the committed changes.