192596 - IMAP Mail gets marked as junk when also reading with Mail.app

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Simon Fraser [no longer active]]

I'm seeing large numbers of mail (like bugmail) get marked as junk in recent
trunk builds. This seems to happen for messagse that I've first seen (and maybe
read) at home, where I use Mail.app on OS X.

Comment 1

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

we use the same imap key word that mail.app uses.

so if mail.app things the message was junk, I think it would set the keyword, 
and then in mozilla, we'd see it as junk too.

or did mail.app not think it was junk?

Comment 2

[Simon Fraser [no longer active]]

Mail.app does not think that any of these are junk.

Comment 3

[Gilles Durys]

*** Bug 201680 has been marked as a duplicate of this bug. ***

Comment 4

[David :Bienvenu]

Simon, can we get a protocol log from Mozilla? thx.

http://www.mozilla.org/quality/mailnews/mail-troubleshoot.html#imap

Comment 5

[David :Bienvenu]

Attachment: proposed fix

With Simon's help, I determined that Mac Mail is using the NotJunk keyword,
which we were parsing as "Junk" because the keyword code wasn't parsing the
keywords into individual keywords. The attached fix checks for NotJunk
explicitly, which has the advantage of not recategorizing mail that Mac Mail
has already categorized.

Comment 6

[David :Bienvenu]

taking

Comment 7

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

*** Bug 206669 has been marked as a duplicate of this bug. ***

Comment 8

[Simon Fraser [no longer active]]

Shouldn't this code be looking for whole words in keywords, not just doing
substring matches?

Comment 9

[David :Bienvenu]

yes, I added a ### TODO comment to that effect in the code - but that's too
risky for 1.4 final. The patch fixes the only known problem and is low risk.

Comment 10

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

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

sr/a=sspitzer for 1.4 final.

thanks to sfraser and hangas for not letting us forget about this, and for
bienvenu for the fix.

Comment 11

[David :Bienvenu]

bug 206786 spun off for remaining tokenizing issue.