Page MenuHomePhabricator
Differential D19083

Support "encoding", which is required by PHPMailerLite, in SES adapter
ClosedPublic
Actions

Authored by epriestley on Feb 14 2018, 6:55 PM.
Tags
None
Referenced Files
F18813469: D19083.id.diff
Mon, Oct 20, 4:44 PM
F18762436: D19083.id45740.diff
Mon, Oct 6, 6:59 PM
F18747301: D19083.id.diff
Fri, Oct 3, 3:30 PM
F18743662: D19083.diff
Fri, Oct 3, 12:25 AM
F18625635: D19083.diff
Sep 15 2025, 11:11 PM
F18566584: D19083.id.diff
Sep 9 2025, 2:44 PM
F18502165: D19083.diff
Sep 4 2025, 10:14 PM
F18249680: D19083.id.diff
Aug 21 2025, 2:50 PM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rP6dd4d870cac2: (stable) Support "encoding", which is required by PHPMailerLite, in SES adapter
rP6bfd0ff275c1: Support "encoding", which is required by PHPMailerLite, in SES adapter
Summary

See https://discourse.phabricator-community.org/t/amazon-ses-adapator-broken-after-upgrade/1121. The adapter class tree is a mess and this property is read by the parent class.

Test Plan

Configured an SES mailer, used bin/mail send-test to reproduce the issue before the patch and observe it working after the patch.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 14 2018, 6:55 PM
Harbormaster completed remote builds in B19543: Diff 45740.Feb 14 2018, 6:56 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 14 2018, 6:56 PM
epriestley requested review of this revision.
Closed by commit rP6bfd0ff275c1: Support "encoding", which is required by PHPMailerLite, in SES adapter (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
metamta/
adapter/
3 lines

Diff 45741

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationAmazonSESAdapter.php

Loading...
Phabricator · Built by Phacility