Differential D18205

Add SNS mail endpoint
Needs ReviewPublic
Actions

Authored by joshuaspence on Jul 11 2017, 1:53 PM.

Tags

None

Referenced Files

	F19934377: D18205.id.diff
	Sat, Apr 11, 3:23 PM

	F19933332: D18205.id.diff
	Fri, Apr 10, 11:41 PM

	F19924204: D18205.id43872.diff
	Tue, Apr 7, 5:48 AM

	F19911743: D18205.diff
	Thu, Apr 2, 2:43 PM

	F19478484: D18205.diff
	Jan 7 2026, 9:06 PM

	F18846153: D18205.id.diff
	Oct 29 2025, 5:11 PM

	F18846117: D18205.id.diff
	Oct 29 2025, 4:45 PM

	F18843605: D18205.diff
	Oct 28 2025, 11:40 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T12910: Allow SES to be used for inbound mail

Summary

Ref T12910. Adds support for receiving inbound mail via SES (using an SNS topic).

Test Plan

This needs a bit of work but I wanted to see whether the upstream was interested in this change before I commit to tidying it up.

Diff Detail

Repository

Branch

master

Lint

Lint Errors

Severity	Location	Code	Message
Error	src/applications/metamta/controller/PhabricatorMetaMTASNSReceiveController.php:39	XHP5	Use of Undeclared Variable
Warning	src/applications/metamta/controller/PhabricatorMetaMTASNSReceiveController.php:145	TXT3	Line Too Long
Warning	src/applications/metamta/controller/PhabricatorMetaMTASNSReceiveController.php:157	TXT3	Line Too Long
Advice	src/applications/metamta/controller/PhabricatorMetaMTASNSReceiveController.php:136	XHP16	TODO Comment

Unit

Tests Passed

Build Status

Buildable 17735
Build 23813: Run Core Tests
Build 23812: arc lint + arc unit

Event Timeline

joshuaspence created this revision.Jul 11 2017, 1:53 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 11 2017, 1:53 PM

Herald added a subscriber: Korvin. · View Herald Transcript

minor

amckinley mentioned this in T12847: A Pathway Towards Private Clusters.Jul 19 2017, 4:41 PM

JoyceBabu added a subscriber: JoyceBabu.Oct 30 2018, 1:13 PM

Revision Contents
Changeset List

Path

Size

scripts/

ssh/

ssh-connect.php

6 lines

src/

__phutil_library_map__.php

2 lines

applications/

metamta/

application/

PhabricatorMetaMTAApplication.php

1 line

controller/

PhabricatorMetaMTASNSReceiveController.php

192 lines

Diff 43872

scripts/ssh/ssh-connect.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/metamta/application/PhabricatorMetaMTAApplication.php

Loading...

src/applications/metamta/controller/PhabricatorMetaMTASNSReceiveController.php

Loading...