314990 - Not using unique message id when msg doesn't have message id

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[David :Bienvenu]

We're not creating a unique message id when an incoming message id doesn't have a message-id. This leads to trouble when we delete duplicate messages, if mailers like Outlook don't deign to put a Message-ID in the message.  Fix upcoming.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

Like 4.x, generate an md5 hash of all the headers.

Comment 2

[Scott MacGregor]

alternatively, we could not delete dupes that don't have message IDs. But your solution is consistent with 4.x.

Does this mean if I get two messages from two different outlook users today that i might lose one in my trash due to my dupe action?

Comment 3

[David :Bienvenu]

we already ignore messages w/o message-id's for the purpose of duplicate deletion. But, the code I changed used to make it so we think we have a non-empty message-id, and it's always the same message id!

Comment 4

[David :Bienvenu]

And yes, sort of, to your Outlook question - basically, if you get two messages from an Outlook client with the same subject, (doesn't matter what the sender is), the second will be deleted if it came in the same session.

This only affects Outlook messages that get sent out onto the internet, I believe - some bizarre claim on MS's part that message-ids are bad somehow.

Comment 5

[David :Bienvenu]

Comment on attachment 201785 [details] [diff] [review]
proposed fix

we'll definitely want this if we add ui for detecting and removing duplicate incoming messages.

Comment 6

[Scott MacGregor]

Comment on attachment 201785 [details] [diff] [review]
proposed fix

approving for the branch too.