218732 - Return receipts contain invalid headers for offline IMAP messages

Status: RESOLVED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[Toby Milne]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.5b) Gecko/20030827
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.5b) Gecko/20030827

It looks like Mozilla Mail adds some internal data to messages stored locally
for an offline IMAP server, and this data is included in return receipts.
Online only messages do not have this data.

Reproducible: Always

Steps to Reproduce:
1. Setup an IMAP account
2. Mark a folder (or the inbox) as an offline folder
3. Send a message with read receipts on, to the mailbox
4. Examine the message source in Mozilla (Ctrl U), or examine the receipt


Actual Results:  
The returned receipt quotes the original message headers in MIME part 3 of the
message, and includes the following data.
--
From - Tue Sep 09 12:36:09 2003
X-Mozilla-Status: 0001
X-Mozilla-Status2: 00000000
--
The first line is missing a colon (:) seperator

Expected Results:  
Messages in folders not marked offline do not have this extra data, so I suppose
the data should not be present, or should be rfc compliant.

Client is a Win 2003 Server
Server is a Linux Box running Courier IMAP

Comment 1

[David :Bienvenu]

the x-mozilla-status headers are used internally,just like they are for local
mail folders. The From header is the same as for local mail folders with the
berkeley mailbox format. It's the message delimiter. As you say, the bug is that
this gets out in the return receipt. In my test, the x-mozilla-status headers
are actually filtered out (there's code to do that; not sure why it didn't do it
in your case), but the "From " delimiter is not filtered.

Comment 2

[David :Bienvenu]

Attachment: proposed fix

filter out "From " header

Comment 3

[Toby Milne]

Correction: I'm only seeing the from line in receipts , the Mozilla Status flags
appear in the view source window (Ctrl U). Sorry for the confusion.

Comment 4

[David :Bienvenu]

fix checked into 1.6