26735 - IMAP mailbox state ignored

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[Richard Zach]

Trying to compact an IMAP folder, I got an alert saying:

Read-only Mailbox! Your browser is not RFC compliant since it ignores the
mailbox state

The weird thing is that I can move messages in and out of folders, and can mark
them deleted, so it's not really read-only.  So this may well be a server
problem.  I'm attaching the IMAP log just in case there is something here.

Linux build 2000.02.05.08

Comment 1

[Richard Zach]

Attachment: IMAP log

Comment 2

[jefft]

Yup, it's a read only mailbox shouldn't allow compact to that folder.

Comment 3

[selmer (gone)]

Mass moving to M16 to get these off the M15 radar.  Please let me know if this
is really an M15 stopper.

Comment 4

[selmer (gone)]

Mass moving M16 to M17 - look for nsbeta2 before anything else.

Comment 5

[scottputterman]

Moving to M18 and nominating for beta3.

Comment 6

[lchiang]

- per mail triage

Comment 7

[jefft]

Attachment: a fix

Comment 8

[jefft]

Can I get a r= & sr=?

Comment 9

[David :Bienvenu]

OK, sr=bienvenu, but we shouldn't close the bug, just reassign it to me - there
are many other operations that we should pay attention to the read only state
for besides expunge.

Comment 10

[jefft]

As you wish. I'll let you check in the fix too. Thanks,

Comment 11

[David :Bienvenu]

adding mail3 keyword

Comment 12

[David :Bienvenu]

changing priorities

Comment 13

[scottputterman]

any ideas on which operations or are there so many of them it's not worth listing?

Comment 14

[David :Bienvenu]

this just needs to be done in a centralized place, probably
ProcessSelectedStateUrl, before getting to the individualized handlers for imap
commands. I believe 4.x handled this correctly. It's not hard.

Comment 15

[scottputterman]

I know it's probably easy, but what are some of the more important operations
that are affected if we don't fix this?

Comment 16

[David :Bienvenu]

deleting messages, marking messages read, compacting the folder, copying
messages into the folder - all of these will cause an alert

Comment 17

[scottputterman]

marking nsbeta1+ and moving to mozilla0.8

Comment 18

[scottputterman]

moving to mozilla0.9

Comment 19

[scottputterman]

marking nsbeta1-

Comment 20

[David :Bienvenu]

fix checked in - we pay attention to the read-only state now and don't issue the
protocol that causes the alert. Richard, if you're still around, it would be
great if you could verify this since I doubt our QA has access to any read-only
mailboxes. Thanks!

Comment 21

[Karen Huang]

Richard, can you help for verifying this bug since we don't have access to any
read-only mailboxes. Please mark as verified if problem has be fixed.
Reopen if problem still exists. Thanks!

Comment 22

[Karen Huang]

Adding "need help for verify" on the status whiteboard.

Comment 23

[Karen Huang]

Again....
Richard, can you help for verifying this bug ? Please mark as verified if
problem has be fixed. Thanks!

Comment 24

[Richard Zach]

As far as I can tell, this is fixed.