Closed
Bug 669925
Opened 13 years ago
Closed 13 years ago
Wrong "From" shown in message list when email address contains Unicode
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(thunderbird10+ fixed, thunderbird11 fixed)
RESOLVED
FIXED
Thunderbird 12.0
People
(Reporter: alexander.buchner, Assigned: standard8)
References
Details
Attachments
(3 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Bienvenu
:
review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:5.0) Gecko/20100101 Firefox/5.0
Build ID: 20110615151330
Steps to reproduce:
I opened my spam folder. I saw a mail, which has a name in the "From"-column, that I know (I also have an address book entry to this name, but without an eMail address). This name was not shown in the from-field of the preview.
The name also doesn't occur in the message source.
Actual results:
There is a name shown in the "From"-column, which is not part of the source.
Expected results:
"From" of message list and message preview should be the same.
No values should appear that aren't part of the source.
As this a spam mail, some users give more credibility, if they "know" the sender...
Reporter | ||
Comment 1•13 years ago
|
||
Comment 2•13 years ago
|
||
Does rebuilding the index for the folder that contains the email helps ?
Reporter | ||
Comment 3•13 years ago
|
||
No, it doesn't.
Comment 4•13 years ago
|
||
I'm guessing this has to do with the fact that the email address is non-ASCII and the display name lookup from bug 243631 is barfing on that, but I'm not totally sure.
Comment 5•13 years ago
|
||
(In reply to comment #4)
> I'm guessing this has to do with the fact that the email address is
> non-ASCII and the display name lookup from bug 243631 is barfing on that,
> but I'm not totally sure.
I have the same problem in some newsgroups and in all cases the email adresses in the From column contains non-ASCII characters.
Reporter | ||
Comment 6•13 years ago
|
||
I just noticed three things:
1. You can simply find this kind of mail by sorting by "From", because the mails get really sorted by the alias in the Source (in my example "Dr. Friedelinde.Friederike", so it will appear at "D") but as stated before in the overview you see the entry of your address book. So these eMails seem to be sorted in the wrong way.
2. Today the exact mail I mentioned before, has a different "From", also from my address book.
3. Both entries of the address book are entries without an email address. Perhaps this helps, perhaps it's just coincidence.
Comment 7•13 years ago
|
||
Bug confirmed when an user with domain @Fjørndættehotmail.com (a mangled version of adress obfucation was seen by several thunderbird5-users with a "faked" name taken from their adressbook.
When the sender removed the non-ascii characters, to @Fjrndttehotmail.com, the sender name appeared correctly.
See for instance Message-ID: <4e30703b$0$311$14726298@news.sunsite.dk> from dk.edb.internet.software.mail+news where thunderbird5 does not show my name as sender.
If you uncheck "Show only display name for people in my address book" the right sender name is shown.
You find it here:
Tools > Options > Advanced > click the Read and display tab.
But still this must be a bug.
Comment 10•13 years ago
|
||
Confirming this, since I see it too. It appears to pick up the first contact in the address book that matches the substring before the first Unicode character, e.g. Bärb@example.com would use the display name from Bob@nowhere.com in the address book.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Wrong "From" shown in message list. → Wrong "From" shown in message list when email address contains Unicode
Comment 11•13 years ago
|
||
WADA, this bug and bug 672966 aren't actually dependent on one another (though they're both the result of the same change). This is an actual bug, in that we don't handle Unicode properly when looking for a contact in the address book. Bug 672966 is just an issue with user confusion, which can hopefully be helped with some documentation.
No longer depends on: 672966
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #544518 -
Attachment mime type: application/octet-stream → text/plain
Comment 13•13 years ago
|
||
(In reply to Jim Porter (:squib) from comment #10)
> It appears to pick up the first contact
> in the address book that matches the substring before the first Unicode
> character
I cannot see this connection here. It is always the same contact from the address book. But in my case the from header contains no email address, but only a name with a non-ASCII character.
I made some tests:
»From: Big Bad Spammer« → TB shows “Big Bad Spammer”
»From: Big Bäd Spammer« → TB shows contact form the address book
»From: Big Bäd Spammer a@b.invalid« → TB shows contact from the address book
»From: Big Bäd Spammer <a@b.invalid>« → TB shows “Big B�d Spammer”
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → mbanner
tracking-thunderbird10:
--- → ?
Assignee | ||
Comment 15•13 years ago
|
||
This is a two part fix:
1) In nsAbMDBDirectory we now check to see if the email address has been truncated after conversion of utf-8 to utf-16, if so, there's probably been an error, so return the fact (we already check the email address isn't empty at the start of the function).
2) In the DB view, use the decoded form of the from field message header rather than the encoded form. This means we get a proper utf-8 value passed to GetCardForEmailAddress and we display the right thing in the thread list.
Attachment #587443 -
Flags: review?(dbienvenu)
Comment 16•13 years ago
|
||
Comment on attachment 587443 [details] [diff] [review]
The fix
this looks good - I'll try to test it later.
Attachment #587443 -
Flags: review?(dbienvenu) → review+
Assignee | ||
Comment 17•13 years ago
|
||
Comment on attachment 587443 [details] [diff] [review]
The fix
Checked into trunk:
http://hg.mozilla.org/comm-central/rev/4a3f5d039f95
Attachment #587443 -
Flags: approval-comm-beta+
Attachment #587443 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 18•13 years ago
|
||
Checked into branches:
http://hg.mozilla.org/releases/comm-aurora/rev/2f1c07cca30f
http://hg.mozilla.org/releases/comm-beta/rev/f80720b2f49c
Status: NEW → RESOLVED
Closed: 13 years ago
status-thunderbird10:
--- → fixed
status-thunderbird11:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 12.0
You need to log in
before you can comment on or make changes to this bug.
Description
•