Page MenuHomePhabricator

Move commits to the "COMMIT" mail prefix
ClosedPublic

Authored by epriestley on Sep 25 2015, 3:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 3:54 PM
Unknown Object (File)
Sat, Dec 14, 7:27 AM
Unknown Object (File)
Thu, Dec 12, 2:22 AM
Unknown Object (File)
Sun, Dec 8, 8:19 AM
Unknown Object (File)
Fri, Dec 6, 12:56 PM
Unknown Object (File)
Tue, Dec 3, 7:20 AM
Unknown Object (File)
Wed, Nov 27, 1:09 PM
Unknown Object (File)
Tue, Nov 26, 3:01 AM
Subscribers
None

Details

Summary

Fixes T9427. Currently, replies to audits/commits go to "Cxxx", but so do replies to countdowns.

There is non real non-disruptive approach available here and this seems least-bad.

Test Plan
  • Made a comment on a commit.
  • Fished the reply-to address out of bin/mail list-oubound + bin/mail show-outbound (it was now "COMMIT...").
  • Sent mail to that address.
  • Grabbed the raw message and wrote it to mail.txt.
  • Ran cat mail.txt | ./scripts/mail/mail_handler.php --process-duplicates.
  • Used bin/mail list-inbound + bin/mail show-inbound to verify receipt.
  • Saw comment appear on audit.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move commits to the "COMMIT" mail prefix.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.EditedSep 25 2015, 3:12 PM
chad edited edge metadata.

There is non real non-disruptive approach available here and this seems least-bad.

wut

This revision is now accepted and ready to land.Sep 25 2015, 3:12 PM

Oh, haha.

There is non real non

Should be "There is no real non-disruptive...": we have to break something since the current state of the world doesn't work.

This revision was automatically updated to reflect the committed changes.