Closed
Bug 239181
Opened 21 years ago
Closed 20 years ago
IMAP STATUS command should not be used on selected mailbox
Categories
(MailNews Core :: Networking: IMAP, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emaijala+moz, Assigned: Bienvenu)
Details
(Whiteboard: fixed-aviary1.0)
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
Running trunk build 2004032208 on WinXP. The server is running uw-imapd.
I was logging my IMAP traffic and noticed that Mozilla tries to query status of
a selected folder. In a few minutes I got quite a few lines like this:
1256[248d208]: 3475890:atp.fi:S-Mail/SPAM:CreateNewLineFromSocket: * NO CLIENT
BUG DETECTED: STATUS on selected mailbox: Mail/SPAM
According to the log file, the connection had Mail/SPAM already selected and it
had been IDLE before that (previously issued commands were IDLE and DONE).
RFC 3501 states, that "the STATUS command SHOULD NOT be used on the currently
selected mailbox".
Setting severity to major as this could cause severe problems with some servers.
|
Reporter | |
Updated•21 years ago
|
Flags: blocking1.7?
|
Assignee | |
Comment 1•20 years ago
|
||
|
|
Assignee | |
Comment 2•20 years ago
|
||
Comment on attachment 149082 [details] [diff] [review]
use NOOP instead of STATUS if we're selected
I think this will do the right thing when new messages come in, but it would be
great if you could try it out, Ere. I'm pretty sure we won't be doing STATUS on
selected folders, and * think we'll update the counts for unread and recent
correctly.
Attachment #149082 -
Flags: superreview?(mscott)
|
||
Updated•20 years ago
|
Attachment #149082 -
Flags: superreview?(mscott) → superreview+
|
|
Assignee | |
Comment 3•20 years ago
|
||
Attachment #149082 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 4•20 years ago
|
||
Comment on attachment 149102 [details] [diff] [review]
whoops, correct fix.
that was a patch from a different tree, for a different bug...
Attachment #149102 -
Flags: superreview?(mscott)
|
|
||
Updated•20 years ago
|
Attachment #149102 -
Flags: superreview?(mscott) → superreview+
|
|
Assignee | |
Comment 5•20 years ago
|
||
fixed on trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
||
Updated•20 years ago
|
Whiteboard: fixed-aviary1.0
|
||
Updated•20 years ago
|
Flags: blocking1.7? → blocking1.7-
|
|
Assignee | |
Comment 6•20 years ago
|
||
re-opening, because I'm going to back out this fix. It caused the regression
http://bugzilla.mozilla.org/show_bug.cgi?id=245044?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
Assignee | |
Comment 7•20 years ago
|
||
fixed by fix in bug 245769
Status: REOPENED → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: MailNews → Core
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•