Differential D10505

Set up support for SES API hosts.
AbandonedPublic
Actions

Authored by epriestley on Sep 17 2014, 3:37 AM.

Tags

None

Referenced Files

	F14014485: D10505.diff
	Sun, Nov 3, 5:20 AM

	F14005631: D10505.diff
	Sun, Oct 27, 5:17 PM

	F13970827: D10505.diff
	Thu, Oct 17, 10:01 AM

	Unknown Object (File)
	Fri, Oct 11, 11:39 AM

	Unknown Object (File)
	Sep 12 2024, 6:25 AM

	Unknown Object (File)
	Sep 5 2024, 9:53 AM

	Unknown Object (File)
	Sep 4 2024, 3:22 PM

	Unknown Object (File)
	Sep 4 2024, 10:41 AM

Subscribers

Tokens

"Like" token, awarded by arcdigital.

Details

Reviewers

• disciplezero

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T5116: Amazon SES mailer does not allow specifying endpoint

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

Add amazon-ses.host config value. It is currently defaulted to the same value it would have been if unset, so it should not affect existing users.

Test Plan

Tried with multiple values (default and west coast).

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2554
Build 14024: Run Core Tests
Build 2558: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

• disciplezero updated this revision to Diff 25242.Sep 17 2014, 3:37 AM

• disciplezero retitled this revision from to Set up support for SES API hosts..

• disciplezero updated this object.

• disciplezero edited the test plan for this revision. (Show Details)

• disciplezero added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 17 2014, 3:37 AM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B2554: Diff 25242.Sep 17 2014, 3:38 AM

• disciplezero added a task: T5116: Amazon SES mailer does not allow specifying endpoint.Sep 17 2014, 3:39 AM

写得很好

conf/default.conf.php
337	写得不错

mikn added a subscriber: mikn.Jun 22 2015, 7:50 PM

@epriestley are you open to taking this? (Ran into this issue because I got SES set up on the west coast and Amazon makes you contact support if you want to send email from new regions :/)

@disciplezero are you still maintaining this? or should i commandeer it?

conf/default.conf.php
337	Probably should be "amazon-ses.endpoint" for consistency and set to null here.
src/applications/config/option/PhabricatorAWSConfigOptions.php
26	See line 39 for indentation style

john.sandall added a subscriber: john.sandall.Sep 30 2015, 4:11 PM

john.sandall mentioned this in T5116: Amazon SES mailer does not allow specifying endpoint.Oct 2 2015, 4:31 PM

arcdigital awarded a token.Feb 23 2016, 10:58 PM

arcdigital added a subscriber: arcdigital.

eadler mentioned this in D15515: Add `amazon-ses.endpoint` configuration.Mar 25 2016, 7:42 PM

Obsoleted by D15515.

Revision Contents
Changeset List

Path

Size

conf/

default.conf.php

1 line

src/

applications/

config/

option/

PhabricatorAWSConfigOptions.php

8 lines

metamta/

adapter/

PhabricatorMailImplementationAmazonSESAdapter.php

3 lines

docs/

user/

configuration/

configuring_outbound_email.diviner

2 lines

Diff 25242

conf/default.conf.php

Loading...

src/applications/config/option/PhabricatorAWSConfigOptions.php

Loading...

src/applications/metamta/adapter/PhabricatorMailImplementationAmazonSESAdapter.php

Loading...

src/docs/user/configuration/configuring_outbound_email.diviner

Loading...