Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Commit

Permalink
fix(sms): Use the real email sender when sending via MockNexmo
Browse files Browse the repository at this point in the history
We were using a made up sender, which caused our email provider
to drop the messages on the ground when testing on fxa-ci. Messages
were reported as delivered, but never seen again.
  • Loading branch information
Shane Tomlinson committed Jun 7, 2017
1 parent ae0a4f8 commit 577db70
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/mock-nexmo.js
Expand Up @@ -39,7 +39,8 @@ function MockNexmo(log, config) {

// HACK: Enable remote tests to see what was sent
mailer.sendMail({
from: `sms.${senderId}@restmail.net`,
from: config.smtp.sender,
sender: config.smtp.sender,
to: `sms.${phoneNumber}@restmail.net`,
subject: 'MockNexmo.message.sendSms',
text: message
Expand Down

0 comments on commit 577db70

Please sign in to comment.