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
F15532977: D19083.diff
Wed, Apr 23, 5:49 PM
F15507370: D19083.id45740.diff
Tue, Apr 15, 7:05 PM
F15489889: D19083.id45741.diff
Fri, Apr 11, 1:11 PM
F15487516: D19083.id.diff
Thu, Apr 10, 2:47 PM
F15485460: D19083.diff
Wed, Apr 9, 11:53 PM
F15478519: D19083.id45740.diff
Tue, Apr 8, 12:30 AM
F15416542: D19083.id45741.diff
Mar 20 2025, 11:53 AM
F15393668: D19083.diff
Mar 15 2025, 10:59 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