266107 - DATE header misparsed: "Mar" (Day of Week) interpreted as "March", "Oct" field is ignored

Status: NEW

(MailNews Core :: Backend, defect)

Description

[Thomas Bertels]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; rv:1.7.3) Gecko/20041001 Firefox/0.10.1
Build Identifier: Mozilla Thunderbird 0.8 (20040913)(Windows; U; Windows NT 5.1; rv:1.7.3)

I've received a message today (26/10/2004 in DD/MM/YYYY) and Thunderbird says me
it was sent the 26/10/2004. The problem comes from the source code: the DATE
attribute is "DATE: mar., 26 oct 2004 06:24:08". It seems it has been bad
generated: "mar." has been used for the day Tuesday instead of "Tue". But
Thunderbird should now that the DATE attribute must be interpreted as [day
short], [n°of day] [month short] [year] [hour]. Thunderbird believe "mar." is
the month but it can't, it's just the day in italian (martedì).

Reproducible: Didn't try
Steps to Reproduce:

Comment 1

[Mike Cowperthwaite]

(In reply to comment #0)
> I've received a message today (26/10/2004 in DD/MM/YYYY) and Thunderbird
> [tells] me it was sent the 26/10/2004.
                             ^^^^^^^^^^  -- what did you actually mean here?

Comment 2

[Thomas Bertels]

I mean the 26/10/2004 (DD/MM/YYYY).

Comment 3

[Thomas Bertels]

Sorry, it's a mistake.
I mean the 26/03/2004 (DD/MM/YYYY).
             ^^^^  --- (March)

Comment 4

[Mike Cowperthwaite]

The date is also misparsed if the period is left off the day abbreviation:
  Date: mar, 26 oct 2004 06:24:08

The comma ought to be sufficient to indicate that the "mar" should be discarded.

Reproduced with TB 1.0 and Moz 1.8a6, Win2K.

Comment 5

[(not reading, please use [e-mail] instead)]

sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.

Comment 6

[Wayne Mery (:wsmwk)]

Thomas, does this still reproduce for you?

Comment 7

[Thomas Bertels]

Attachment: testcase.eml

Yes, still reproduceable on latest nightly.

Here's a minimized testcase.

Comment 9

[chrizilla]

see additional testcase in bug 462033:
https://bugzilla.mozilla.org/attachment.cgi?id=345141

Comment 10

[Thomas Bertels]

See also bug 462033 comment 6
Current link to code: https://searchfox.org/mozilla-central/source/nsprpub/pr/src/misc/prtime.c#945 (PR_ParseTimeStringToExplodedTime)

The code in question is most likely

case 'm': case 'M':
   if (month == TT_UNKNOWN &&
      (rest[1] == 'a' || rest[1] == 'A') &&
      (rest[2] == 'r' || rest[2] == 'R')) {
      month = TT_MAR;
   }