Differential D19027

Improve correctness of email address escaping in Mailgun/Postmark
ClosedPublic
Actions

Authored by epriestley on Feb 8 2018, 5:00 PM.

Tags

None

Referenced Files

	F15448678: D19027.id45625.diff
	Fri, Mar 28, 6:15 AM

	F15439522: D19027.id.diff
	Wed, Mar 26, 7:55 AM

	F15434419: D19027.diff
	Tue, Mar 25, 3:31 AM

	F15395860: D19027.diff
	Sun, Mar 16, 10:14 AM

	F15394081: D19027.id45625.diff
	Sat, Mar 15, 11:19 PM

	F15394074: D19027.id45624.diff
	Sat, Mar 15, 11:19 PM

	F15394068: D19027.id.diff
	Sat, Mar 15, 11:19 PM

	F15394062: D19027.diff
	Sat, Mar 15, 11:19 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13053: Plans: Mail Tags and Failover
Commits: rP942b17a98088: Improve correctness of email address escaping in Mailgun/Postmark

Summary

Ref T13053. Uses the changes in D19026 to escape mail addresses. Those rules may not be right yet, but they're at least all in one place, have test coverage, and aren't obviously incorrect.

Test Plan

Will vet this more extensively when re-testing all mailers.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 8 2018, 5:00 PM

Harbormaster completed remote builds in B19419: Diff 45624.Feb 8 2018, 5:01 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2018, 5:01 PM

epriestley requested review of this revision.

Closed by commit rP942b17a98088: Improve correctness of email address escaping in Mailgun/Postmark (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

adapter/

PhabricatorMailImplementationAdapter.php

5 lines

PhabricatorMailImplementationMailgunAdapter.php

9 lines

Diff 45625

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationAdapter.php