fix(email): include data from headers in email sent events

mozilla · Oct 16, 2018 · 28a4a53 · 28a4a53
1 parent 87a22be
commit 28a4a53
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/senders/email.js b/lib/senders/email.js
@@ -360,7 +360,7 @@ module.exports = function (log, config) {
               emailService
             })
 
-            emailUtils.logEmailEventSent(log, message)
+            emailUtils.logEmailEventSent(log, Object.assign({}, message,  { headers }))
 
             return d.resolve(status)
           })

diff --git a/test/local/senders/email.js b/test/local/senders/email.js
@@ -844,6 +844,7 @@ describe(
             assert.equal(emailEventLog.args[0].flow_id, 'wibble', 'logs flow id')
             assert.equal(emailEventLog.args[0].template, 'verifyLoginEmail', 'logs correct template')
             assert.equal(emailEventLog.args[0].type, 'sent', 'logs correct type')
+            assert.equal(emailEventLog.args[0].locale, 'en')
             const mailerSend1 = mockLog.info.getCalls()[1]
             assert.equal(mailerSend1.args[0].op, 'mailer.send.1', 'logs mailer.send.1')
             assert.equal(mailerSend1.args[0].to, message.email, 'logs sender to email address')