From 3910799683af4c825b2fb248b4ed9ede556f856e Mon Sep 17 00:00:00 2001 From: Juan Tejada Date: Thu, 15 Dec 2016 01:15:51 -0800 Subject: [PATCH] [local-sync] Fix contact parsing from T7327 Summary: See description at T7327 Test Plan: Manual, but this should have unit tests Reviewers: mark, evan Reviewed By: evan Differential Revision: https://phab.nylas.com/D3513 --- packages/local-sync/src/shared/message-factory.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/local-sync/src/shared/message-factory.js b/packages/local-sync/src/shared/message-factory.js index 59d3b9cde..0c93f3819 100644 --- a/packages/local-sync/src/shared/message-factory.js +++ b/packages/local-sync/src/shared/message-factory.js @@ -24,9 +24,14 @@ function extractContacts(input) { const s = `["${input[0].replace(/"/g, '\\"').replace(/, /g, '", "')}"]`; const values = JSON.parse(s); return values.map(v => { - const {name, address: email} = mimelib.parseAddresses(v).pop() + const parsed = mimelib.parseAddresses(v) + if (!parsed || parsed.length === 0) { + return null + } + const {name, address: email} = parsed.pop() return {name, email} }) + .filter(c => c != null) } /*