866996 - [email] update addressparser node-dep to not think apostrophes are trying to quote display names

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Andrew Sutherland [:asuth] (he/him)]

Just saw an e-mail with John O'Duinn on the CC list where the display names were not otherwise quoted (bad Thunderbird!).  The addressparser version we are using got confused by this and included all of the names in the cc list after him as part of his display name.

addressparser already has a fix and we should take it.

Comment 1

[Andrew Sutherland [:asuth] (he/him)]

I would like this to land on v1-train since v1-train is still taking things and people with apostrophes in their names are people too.

landed in gaia-email-libs-and-more/master:
https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/192
https://github.com/mozilla-b2g/gaia-email-libs-and-more/commit/66400cbdd583a58626cc9b57668f2fb6938f0e17

landed in gaia/master:
https://github.com/mozilla-b2g/gaia/pull/9469
https://github.com/mozilla-b2g/gaia/commit/b9583de0462c6b126921d317bc9a36c3ab264605

Comment 2

[Lukas Blakk [:lsblakk] use ?needinfo]

Small fix, good impact, let's get this uplifted to all branches.

Comment 3

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted b9583de0462c6b126921d317bc9a36c3ab264605 to:
v1-train: ec3ae3fd2c781212594e70989c54cd3faaa35724
v1.0.1: a1d7708fbcda26204a3ca6f04ac89981aa985ffb

Comment 4

[Jeni]

Issue repros in
Inari Build ID: 20130503070205
Kernel Date: Feb 21
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/3f3489356bbc
Gaia: 3e232bce289c9e156d92553e752616cba284bc8f

And in

Unagi Build ID: 20130503070204
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/8becaf2a0bc7
Gaia: b0aca0dd1e2955e11190ede725e1fb9ee596438b

A email from someone who has a apostrophe in their name is displaying as listed