"My Name" from account is ignored when email contains "special chars", full name extracted from email address

RESOLVED FIXED

Status

MailNews Core
Backend
RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: Martin Kutschker, Assigned: Christian Eyrich)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-aviary1.0)

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20030208 Netscape/7.02
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20030208 Netscape/7.02

Settings
Your name: Martin T. Kutschker
Email address: Martin-no$pam-Kutschker@blackbox.n0spam.net


From: "Martin-no$pam-Kutschker" <"Martin-no$pam-Kutschker"@blackbox.n0spam.net>

Reproducible: Always
Steps to Reproduce:
Post any message on a news server.
Actual Results:  
From: "Martin-no$pam-Kutschker" <"Martin-no$pam-Kutschker"@blackbox.n0spam.net>

Expected Results:  
From: "Martin T. Kutschker" <"Martin-no$pam-Kutschker"@blackbox.n0spam.net>

Ok. the $ in the obfuscation address is fancy, but that's no excuse for ignoring
"Your Name".

Comment 1

14 years ago
Martin, from your user agent, you are using Netscape Communicator, and not
Mozilla.  While Netscape is based on Mozilla, bugs in Netscape should be
reported to them, and not here.  See for example:

http://channels.netscape.com/ns/browsers/faq.jsp#gen7

On how to report problems with Netscape.

Your version is also rather old, so the problem may well be fixed in Netscape
7.1, or of course in Mozilla (go to www.mozilla.org to download).

I'm afraid this will be marked INVALID
(Reporter)

Comment 2

14 years ago
I've seen this also on Mozilla (at least 1.5) on Windows. So it is not a
Netscape problem. I never noticed before until I tried those funky address.

Comment 3

14 years ago
Okay, I can reproduce this with a current ThunderBird build (20040426) on Win2k,
but only if the e-mail address contains a $ (I guess other special chars as
well).  When the e-mail address is okay, "your name" is used.

I suppose the bug desciption should be something like:

Your name / real name is ignored when e-mail address contains special characters
(Reporter)

Comment 4

14 years ago
Summary changed as per request of comment #3 in conformance with initial bug report.
Summary: "My Name" from account is ignored, full name extracted from email address → "My Name" from account is ignored when email contains "special chars", full name extracted from email address
(Assignee)

Comment 5

14 years ago
Confirmed with Mozilla 20040401 on Win98.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
(Assignee)

Comment 6

14 years ago
Created attachment 147396 [details] [diff] [review]
proposed patch

It happens to all addresses with a quoted local part, that is
<"localpart"@domain.tld> (those addresses are allowed by RFC 2822).

I tracked this down to construct_addresslist() and while a flag there is called
"quoted", I think it refers to a quoted realname and doesn't know anything of
quoted-string in the local-part.

While I'm not fully understand the function and I couldn't test all possible
variations to write an address, this patch should do it.
It just disables collecting the displayname from within the addr-spec.
(Assignee)

Updated

14 years ago
Assignee: sspitzer → ch.ey
Status: NEW → ASSIGNED
(Assignee)

Comment 7

14 years ago
Who's best to have a look at this code? Scott, any idea?
Component: Networking: News → Mail Back End

Comment 8

14 years ago
that patch seems ok Christian. Certaiinly enough for us to try out and see how
it works.
(Assignee)

Comment 9

14 years ago
Ok, great. Feel free to tack your review on it and check it in if you like.

Comment 10

14 years ago
I checked this in. Let's see how it tests out. closing for now. will re-open if
it breaks something
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Updated

14 years ago
Whiteboard: fixed-aviary1.0
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.