bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Addition of X-Account-Key header to mbox uses CRLF

RESOLVED FIXED

Status

Thunderbird
General
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Mark Peek, Assigned: Bienvenu)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-aviary1.0)

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-us) AppleWebKit/125.2 (KHTML, like Gecko) Safari/125.8
Build Identifier: 0.7+

The addition of the X-Account-Key header for the global inbox feature adds a CRLF to the end of the 
line. All other lines in the mbox file are terminated with just a LF. This can cause compatibility problems 
with other programs that read mbox files.

Reproducible: Always
Steps to Reproduce:
1. Download mail into a global inbox enabled system.
2. Look at the Inbox file in an editor (vi or emacs will do)
3. Notice that the X-Account-Key line contains a "^M" 
Actual Results:  
The Inbox showed this:

From - Mon Jul  5 09:11:34 2004
X-Account-Key: account3^M
X-UIDL: 40859b052c70fedb


Expected Results:  
When it should have had this:

From - Mon Jul  5 09:11:34 2004
X-Account-Key: account3
X-UIDL: 40859b052c70fedb
(Reporter)

Comment 1

14 years ago
Created attachment 152345 [details] [diff] [review]
Patch for CRLF in mbox X-Account-Key header

Comment 2

14 years ago
sending over to David for analysis
Assignee: mscott → bienvenu
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 3

14 years ago
Comment on attachment 152345 [details] [diff] [review]
Patch for CRLF in mbox X-Account-Key header

yes, that's right - I'll check this in...
Attachment #152345 - Flags: superreview+
(Assignee)

Comment 4

14 years ago
fixed, thx for the patch.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Whiteboard: fixed-aviary1.0
You need to log in before you can comment on or make changes to this bug.