Paths

Table of Contentst

Differential D19970

Fix an issue where "CC"-only email improperly wiped CC addresses
ClosedPublic
Actions

Authored by epriestley on Jan 14 2019, 9:32 PM.

Tags

None

Referenced Files

	F18736963: D19970.id47668.diff
	Wed, Oct 1, 8:26 AM

	F18569150: D19970.id47708.diff
	Wed, Sep 10, 1:02 AM

	F18569148: D19970.id47668.diff
	Wed, Sep 10, 1:02 AM

	F18566798: D19970.id.diff
	Tue, Sep 9, 2:54 PM

	F18517719: D19970.diff
	Fri, Sep 5, 1:50 PM

	F18510979: D19970.diff
	Fri, Sep 5, 5:59 AM

	F18099033: D19970.id47708.diff
	Aug 9 2025, 12:49 AM

	F18092503: D19970.id.diff
	Aug 7 2025, 10:57 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T920: Provide SMS Support

Commits

rP96d3e73eed64: Fix an issue where "CC"-only email improperly wiped CC addresses

Summary

Ref T920. See https://discourse.phabricator-community.org/t/ccd-emails-not-working-with-sendgrid-since-2019-week-1-update/2294.

Test Plan

Used bin/mail send-test --cc ... without --to, got email.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 14 2019, 9:32 PM

epriestley added inline comments.Jan 14 2019, 9:33 PM

src/applications/metamta/engine/PhabricatorMailEmailEngine.php
57	This was an incorrect leftover from the old logic which would sometimes move "CC" to "To".

Harbormaster completed remote builds in B21521: Diff 47668.Jan 14 2019, 9:35 PM

epriestley requested review of this revision.Jan 14 2019, 9:35 PM

amckinley accepted this revision.Jan 16 2019, 2:15 AM

This revision is now accepted and ready to land.Jan 16 2019, 2:15 AM

Closed by commit rP96d3e73eed64: Fix an issue where "CC"-only email improperly wiped CC addresses (authored by epriestley). · Explain WhyJan 16 2019, 9:22 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

engine/

PhabricatorMailEmailEngine.php

1 line

Diff 47708

View Options

src/applications/metamta/engine/PhabricatorMailEmailEngine.php

Fix an issue where "CC"-only email improperly wiped CC addressesClosedPublicActions