Fixes T10539. When building mail targets, we fail to preserve whether a recipient was originally "To" or "Cc", and just move everyone to "To".
Details
Details
- Reviewers
chad - Maniphest Tasks
- T10539: Email not respecting TO and CC
- Commits
- Restricted Diffusion Commit
rPba9cd64e514e: Stop moving "Cc" addresses to "To" when building mail targets
Added a comment to a task with a "To" user and a "Cc" user, with metamta.placeholder-to-recipient set and metamta.one-mail-per-recipient set.
Got mail with me Cc'd as the Cc'd user:
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- mtarget1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 11133 Build 13796: Run Core Tests Build 13795: arc lint + arc unit
Event Timeline
Comment Actions
I haven't actually tested this properly yet since my local task queue has like 30K commits to parse from the dummy repository I built in T10368. Once the queue flushes I'll make sure this actually works.