Open Bug 180299 Opened 22 years ago Updated 2 years ago

Support 'In-Reply-To: Your message of "Date" <Message-ID>'

Categories

(MailNews Core :: Database, defect)

defect

Tracking

(Not tracked)

People

(Reporter: u69748, Unassigned)

References

Details

(Keywords: helpwanted, Whiteboard: [patchlove][has draft patch])

Attachments

(1 file, 3 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3a) Gecko/20021111
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3a) Gecko/20021111

Mozilla dose not recognize following type of In-Reply-To header:

In-Reply-to: Your message of "Fri, 08 Nov 2002 18:34:21 JST."
             <Message-ID>

Unix mh mailer generates this, but no References header.
So, Mozilla can't construct thread correctly.
Is this an old header style ?

I receive 200 mails everyday. 1/4 of them are this type of mail.
So, I really want to Mozilla supports this type of header.


Reproducible: Always

Steps to Reproduce:
1. Recieve a reply mail from mh.
2. See the message in thread view mode.

Actual Results:  
thread is broken.


Expected Results:  
correct thread view.
Summary: Support 'In-Repley-To: Your message of "Date" <Message-ID>' → Support 'In-Reply-To: Your message of "Date" <Message-ID>'
same as bug 173486?
If a message has a following header
In-Reply-to: <Message-ID>
there is no problem,
even if the message has no References header.
Attached file sample mbox file (obsolete) —
test4a,test4b,test4c are replies of test3, which have
 In-Reply-To: Your message of "Fri, 08 Nov 2002 18:34:21 JST." <test3>
 In-Reply-To: <test3> Your message of "Fri, 08 Nov 2002 18:34:21 JST."
 In-Reply-To: Your message of "Fri, 08 Nov 2002 18:34:21 JST."
	<test3>
respectively.

Result thread view of 2002111508 trunk is follow:
=test1
|-=test2
  |-=test3
  | |-=test4
  | |-=test4b
  |-=test3a
=test4a
=test4c

Expected result is follow:
=test1
|-=test2
  |-=test3
  | |-=test4
  | |-=test4a
  | |-=test4b
  | |-=test4c
  |-=test3a
This style is defined as obs-in-reply-to in RFC 2822.
Mozilla MUST recognize message-ID from it.

quotation from RFC 2822:
-----------------------
4. Obsolete Syntax

   ...  Though some of these
   syntactic forms MUST NOT be generated according to the grammar in
   section 3, they MUST be accepted and parsed by a conformant receiver.

   ...

4.5.4. Obsolete identification fields

   The obsolete "In-Reply-To:" and "References:" fields differ from the
   current syntax in that they allow phrase (words or quoted strings) to
   appear.  The obsolete forms of the left and right sides of msg-id
   allow interspersed CFWS, making them syntactically identical to
   local-part and domain respectively.

obs-message-id  := "Message-ID" *WSP ":" msg-id CRLF
obs-in-reply-to := "In-Reply-To" *WSP ":" *(phrase / msg-id) CRLF
obs-references  := "References" *WSP ":" *(phrase / msg-id) CRLF
obs-id-left     := local-part
obs-id-right    := domain

   For purposes of interpretation, the phrases in the "In-Reply-To:" and 
   "References:" fields are ignored.

   ...
-----------------------
I'm trying to fix this bug.
Attached patch patch v0.1 (obsolete) — Splinter Review
I think I have fixed this bug.
Please review.
Attached file New sample mbox
Now I can get thread view of test4a - test4j under test3, 
except test4f which has a empty In-Reply-To header.
Attachment #106557 - Attachment is obsolete: true
It seems to me Mozilla should parse this style.
Marking NEW.
Status: UNCONFIRMED → NEW
Ever confirmed: true
QA Contact: olgam → laurel
Attached patch patch v1.0 (obsolete) — Splinter Review
I think I have fixed this bug.
Attachment #107863 - Attachment is obsolete: true
Attachment #109409 - Flags: review?(sspitzer)
Attachment #109409 - Flags: superreview?(bienvenu)
this is the right thing to do in general - I'm not sure that the patch is the
best way to do this - I'll need to look at it more.
Bienvenu,
could you explain what is wrong about this patch?
If possible I will try to fix the problem.

This bug makes my job ability down approximately 5% :-)
Comment on attachment 109409 [details] [diff] [review]
patch v1.0

Bienvenu, Seth,
please comment this patch.
Attachment #109409 - Flags: superreview?(bienvenu)
Attachment #109409 - Flags: review?(sspitzer)
Attachment #109409 - Flags: review?(sspitzer)
*** Bug 173486 has been marked as a duplicate of this bug. ***
*** Bug 173486 has been marked as a duplicate of this bug. ***
Bug 191726 is related to this, if not a dupe.
Product: Browser → Seamonkey
Assignee: sspitzer → mail
Hideyuki EMURA: if you still want this, can you provide an updated patch? Ask Bienvenu for review as Seth doesn't do mail stuff much these days.
OS: Windows XP → All
Hardware: PC → All
QA Contact: laurel
I believe this is a Core mailnews problem, but the patch implies it should be in Database whereas the name implies in MIME...
is this wanted for Thunderbird?
Keywords: helpwanted
Whiteboard: [patchlove]
Assignee: mail → nobody
Component: MailNews: Message Display → Database
Product: SeaMonkey → MailNews Core
QA Contact: database
Whiteboard: [patchlove] → [patchlove][has draft patch]
Comment on attachment 109409 [details] [diff] [review]
patch v1.0

Patch has bitrotted. Hideyuki, up for a new patch? :)

$ patch -p0 --dry-run < ~/Desktop/p180299.diff 
(Stripping trailing CRs from patch.)
patching file public/nsMsgHdr.h
Hunk #1 FAILED at 73.
1 out of 1 hunk FAILED -- saving rejects to file public/nsMsgHdr.h.rej
(Stripping trailing CRs from patch.)
patching file src/nsMsgHdr.cpp
Hunk #1 FAILED at 735.
patch unexpectedly ends in middle of line
Hunk #2 FAILED at 776.
2 out of 2 hunks FAILED -- saving rejects to file src/nsMsgHdr.cpp.rej
Attachment #109409 - Attachment is obsolete: true
Attachment #109409 - Flags: review?(sspitzer)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: