Ref T920. We currently embed the Twilio PHP API, but can replace it with about 100 lines of code and get a future-oriented interface as a bonus. Add a Future so we can move toward a simpler calling convention for the API.
Details
Details
- Reviewers
amckinley - Maniphest Tasks
- T920: Provide SMS Support
- Commits
- rP7e87d254ab33: Add a parameterized Future for Twilio API calls
Used this future to send SMS messages via the Twilio API.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- mfa3
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 21453 Build 29213: Run Core Tests Build 29212: arc lint + arc unit
Event Timeline
Comment Actions
This is a Good Thing regardless of how we go forward, but FWIW, AWS also supports sending SMS via SNS: https://aws.amazon.com/sns/sms-pricing/
Comment Actions
I was looking around for a second provider but didn't realize SNS supported it. "Plivo" seems like another one, do you know of any more? I'd ideally like to aim for ~3 reasonably sensible providers so we can do fallback. Googling hits a lot of stuff but I've never heard of any of it.
Comment Actions
I haven't heard of any of these guys except for AWS and Twilio, which seems like a pretty good combo for fallback. It's probably not worth changing our existing queueing/retry behavior, but FWIW SNS makes pretty serious resilience promises:
Upon receiving a publish request, SNS stores multiple copies (to disk) of the message across multiple Availability Zones before acknowledging receipt of the request to the sender.
Also just for giggles, I'm going to subscribe our phones via SMS to the AWS CloudWatch alarm that actually has caught a few real issues (as opposed to the strange "1/4 of a machine is down" alarms).
Comment Actions
Groovv Offers is a text, email and social marketing solution by Total Merchant Services. Groovv Offers acquired Fanminder, one of the SMS text messaging services included on this original list. Groovv does not make pricing public on its website.
Wow this provider sounds RELIABLE.