Page MenuHomePhabricator

Add infrastructure for sending SMS via AWS SNS
ClosedPublic

Authored by amckinley on Jan 16 2019, 7:09 PM.
Tags
None
Referenced Files
F15544722: D19982.id47826.diff
Sat, Apr 26, 6:45 AM
F15543067: D19982.id47828.diff
Fri, Apr 25, 8:23 PM
F15537493: D19982.id47824.diff
Thu, Apr 24, 2:07 PM
F15532706: D19982.id47827.diff
Wed, Apr 23, 5:01 PM
F15530164: D19982.id47826.diff
Wed, Apr 23, 2:21 AM
F15529990: D19982.id47828.diff
Wed, Apr 23, 12:12 AM
F15527025: D19982.id47684.diff
Tue, Apr 22, 12:10 AM
F15526096: D19982.id47684.diff
Mon, Apr 21, 7:06 PM
Subscribers

Details

Summary

Ref T920. Ref T13235. This adds a Future, similar to TwilioFuture, for interacting with Amazon's SNS service.

Also updates the documentation.

Also makes the code consistent with the documentation by accepting a media argument.

Test Plan

Clicked the "send test message" button from the Settings UI.

Diff Detail

Repository
rP Phabricator
Branch
sms-redux
Lint
Lint Errors
SeverityLocationCodeMessage
Error/Users/amckinley/src/phacility/core/lib/libphutil/src/filesystem/Filesystem.php:428PHL1Unknown Symbol
Error/Users/amckinley/src/phacility/core/lib/libphutil/src/filesystem/Filesystem.php:568PHL1Unknown Symbol
Unit
Tests Passed
Build Status
Buildable 21678
Build 29563: Run Core Tests
Build 29562: arc lint + arc unit

Event Timeline

Clean up to work like TwilioFuture.

amckinley edited the test plan for this revision. (Show Details)
epriestley added inline comments.
src/applications/metamta/future/PhabricatorAmazonSNSFuture.php
4

Is this actually used?

5

Doesn't necessarily need to be in this diff, but can we put this in PhutilAWSFuture?

src/docs/user/configuration/configuring_outbound_email.diviner
100

(I'll hit this document a bit more broadly for SMS in general at some point, too.)

216

Link to SES instead of SNS?

220

"to the SES configuration"?

src/infrastructure/cluster/config/PhabricatorClusterMailersConfigType.php
48

Ah, thanks.

This revision is now accepted and ready to land.Jan 24 2019, 12:12 AM
amckinley added inline comments.
src/applications/metamta/future/PhabricatorAmazonSNSFuture.php
4

If I implemented getProxiedFuture correctly, I think so?

5

Yeah, good idea.

src/docs/user/configuration/configuring_outbound_email.diviner
216

Hahah I noticed both of these right after submitting.

Oh, I see what's going on with parameters, the base class is just a little bit unusual. Carry on.

This revision was automatically updated to reflect the committed changes.
amckinley marked an inline comment as done.