206791 - imap does not correctly set seen flag unless the user has Write Flag ACL on the mail box. Should only need Seen Flag ACL

Status: RESOLVED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[Matt Lytle]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0; .NET CLR 1.0.3705)
Build Identifier: 1.3.1B

When retrieving messages from an IMAP Server (Cyrus IMAP Server), it appears 
that Mozilla uses body.peek to cache the messages locally.  However, when a 
user reads these messages, they are not marked as seen on the server.  It 
appears that unless the user has the W and S IMAP ACL's on that folder, it will 
not attempt to write the \Seen flag.  It should only require the S ACL for this 
to work correctly.

Reproducible: Always

Steps to Reproduce:
1. Read mail in a folder that the user does not have the W ACL on.
2. Change to another Folder
3. Open the original folder again, the mail will be marked as unread.

Actual Results:  
The mail was marked as unread.

Expected Results:  
The mail should have been properly marked as read.

Comment 1

[David :Bienvenu]

could be we're just checking the W ACL - I'll check.

Comment 2

[David :Bienvenu]

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

Comment 3

[Matt Lytle]

I've tested this with 1.4RC1.  When reading messages individually it seems to 
work fine.  When selecting files and using the "Mark as read option" or using 
the "Mark all as read option", it still does not work.

Thanks for the good work,

--Matt

Comment 4

[David :Bienvenu]

that's odd - I tried this on a cyrus server and didn't have a problem. I didn't
try mark all read, or multiple message mark read, but I will. I don't know why
that should be different, however.

Could you attach an imap protocol log from an attempt to mark multiple messages
read? Thx.

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

Comment 5

[Matt Lytle]

Attachment: IMAP Session logs showing the problem with mark all or mark read

Comment 6

[David :Bienvenu]

the problem is that we're not allowing the ACL settings to override the
[READ-ONLY] property of the folder. I'll need to figure out a way to get access
to the folder's acl info from the IMAP thread so I can check that as well.

Comment 7

[David :Bienvenu]

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

Comment 8

[David :Bienvenu]

Attachment: proposed fix

now I check the acl flags, if any, for the seen flag, and also the permanent
flags, to see if we should be able to manipulate the seen flag, for "read-only"
folders.

Comment 9

[David :Bienvenu]

fix checked in, r/sr=sspitzer

Comment 10

[Matt Lytle]

I justed this on 1.4-RC2, and the bug still occurs when using Mark All as Read 
feature.

--Matt

Comment 11

[David :Bienvenu]

the fix was only checked into 1.5 builds, i.e., trunk builds. Re-resolving
fixed,since it's fixed on the trunk.