Closed Bug 460443 Opened 17 years ago Closed 16 years ago

Thunderbird can't hande correctly multiline subject header that contains CR+HTAB instead of CR+SPC

Categories

(Core :: Networking, defect)

1.9.1 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.3a5

People

(Reporter: eero.volotinen, Assigned: jhorak)

Details

(Keywords: testcase)

Attachments

(3 files)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1) Gecko/2008070400 SUSE/3.0.1-0.1 Firefox/3.0.1 Build Identifier: Thunderbird can't hande correctly multiline subject header that contais CR+HTAB instead of CR+SPC By definition multiline subject line contains: 2. Subject Line Encoding The basic definition of the "Subject:" of an electronic mail message is contained in [RFC2822]. The normative requirements that apply to all headers are: o The maximum length of the header field is 998 characters. o Each line must be no longer than 78 characters. A multi-line subject field is indicated by the presence of a carriage return and white space, as follows: Subject: This is a test Thunderbird works if multiline subject contais CR+SPC, but it bugs and shows square on windows or tab on Linux if smtp client creates standard multiline subject containing CR+HTAB For example groupwise client uses CR+HTAB on multiline subject headers. Eero Reproducible: Always Steps to Reproduce: 1.send multiline subject header contains email to and receive it by thunderbird 2. subjectline is not showed as correct 3. Actual Results: thunderbird does not correctly show multiline subject header Expected Results: thunderbird need to show multiline subject header correctly CR+HTAB need to be handled like CR+SPC because this is defined in http://www.ietf.org/rfc/rfc4096.txt
Steps to Reproduce: 1.send multiline subject header with CR+HTAB email to and receive it by thunderbird 2. subjectline is not showed as correct 3.
Attached patch Initial patch — — Splinter Review
I can fully reproduce with latest Thunderbird (3.0 beta4). Attaching a patch which should fix this issue.
Reproduction with attached mail: 1. save attachement 2. open attached message via File/Open Saved Message... 3. see subject line "...devel perl-..."
Attachment #401819 - Flags: review?(cbiesinger)
Status: UNCONFIRMED → NEW
Component: Mail Window Front End → MIME
Ever confirmed: true
Product: Thunderbird → MailNews Core
QA Contact: front-end → mime
Version: unspecified → 1.9.1 Branch
Keywords: testcase
Comment on attachment 401819 [details] [diff] [review] Initial patch Changing review requestee. Thanks for the patch.
Attachment #401819 - Flags: review?(cbiesinger) → review?(bienvenu)
Flags: in-testsuite?
Component: MIME → Networking
Product: MailNews Core → Core
QA Contact: mime → networking
Attachment #401819 - Flags: review?(bienvenu) → review?(cbiesinger)
Attachment #401819 - Flags: review?(cbiesinger) → review+
(In reply to comment #0) > Thunderbird can't hande correctly multiline subject header that contais CR+HTAB > instead of CR+SPC (Q1) What Tb's behaviour or phenomenon is "Thunderbird can't handle correctly"? (In reply to comment #1) > Steps to Reproduce: > 1.send multiline subject header with CR+HTAB email to and receive it by > thunderbird > 2. subjectline is not showed as correct (Q2) Which case? Both? (a) Subject: text1[CRLF][HTAB]text2[CRLF] (b) Subject: RFC2047-encoded-word-1[CRLF][HTAB]RFC2047-encoded-word-2[CRLF] (Q3) What phenomenon do you mean by "not showed as correct"? (c) [HTAB] in (a) is displayed as U+FFFD at thread pane. (known issue) Display at message header pane is not affected. (displayed as space) (d) If (c), RFC2047 decoding fails, because space between consecutive RFC-2047 encoded words should be 0x20 instead of [HTAB]. (AFAIR, this is also already reported problem, but I'm not sure.) (In reply to comment #2) > Initial patch (Q4) It looks quirks for (d). Is it right?
attached mail is (a). proposed patch works for both (a) and (b)?
Assignee: nobody → jhorak
Status: NEW → ASSIGNED
Attaching type (b) of message. This is even working fine without patch.
Any chance to commit?
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: