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.
Details
Details
- Reviewers
chad - Maniphest Tasks
- T9427: Error Processing Mail (Multiple Receivers) when replying to audit mail
- Commits
- Restricted Diffusion Commit
rP36d9908e6cd1: Move commits to the "COMMIT" mail prefix
- 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
Diff Detail
- Repository
- rP Phabricator
- Branch
- commit1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 8054 Build 9142: [Placeholder Plan] Wait for 30 Seconds Build 9141: arc lint + arc unit
Event Timeline
Comment Actions
There is non real non-disruptive approach available here and this seems least-bad.
wut
Comment Actions
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.