Closed Bug 222394 Opened 16 years ago Closed 13 years ago

IMAP: MYRIGHTS command used on names marked \Noselect

Categories

(MailNews Core :: Networking: IMAP, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Hagedorn, Assigned: Bienvenu)

Details

(Keywords: fixed1.8.0.7, fixed1.8.1)

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.5) Gecko/20031007
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.5) Gecko/20031007

I'm running a Cyrus IMAP server with shared mailboxes. I've set up my IMAP in
account in Mozilla to have the server override the namespaces. So Mozilla finds
out about the shared folders:

<1066289493<3 namespace
>1066289493>* NAMESPACE (("" "/")) (("user/" "/")) (("Shared Folders/" "/"))
3 OK Completed

Then it proceeds to list all mailboxes and gets, among others, these replies:

* LIST (\Noselect \HasChildren) "/" "user"
* LIST (\Noselect \HasChildren) "/" "Shared Folders"

Note that both are marked \Noselect! Still Mozilla sends the following commands
to the IMAP server:

<1066289502<127 myrights "user"
>1066289502>127 NO Invalid mailbox name
<1066289506<128 myrights "Shared Folders"
>1066289506>128 NO Invalid mailbox name

I personally would prefer Mozilla to only to myrights and getacl when a folder
is opened, not preemptively. It definitely should *not* send these commands for
names that can't be selected ...

Reproducible: Always

Steps to Reproduce:
1. Set up Cyrus IMAP
2. Create shared mailboxes
3. Set up Mozilla to list all mailboxes and to use the namespaces specified by
the server

Actual Results:  
There were two error messages "NO Invalid mailbox name", no details about
*which* mailbox names.

Expected Results:  
It shouldn't have tried to do MYRIGHTS on a name for which \Noselect has been
returned.

This is Cyrus IMAP 2.1.12, but I doubt that matters.
The same happens when IMAP server returns a name of folder that doesn't exist 
any more in LSUB response (can happen when I delete a folder, but I don't 
UNSUBSCRIBE it). Mozilla tries GETACL or MYRIGHTS on that folder and shows a 
strange error message to the user: The current command did not succeed: The 
mail server responded: GETACL Cannot access folder.


Product: MailNews → Core
This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/
This bug still exists in Thunderbird 1.5b1. I just checked it.
Just to keep this alive: the bug is still present in TB 1.5 final. It should be rather trivial to fix, but apparently nobody else seems to mind ... Granted, it only happens when you create the account for the first time, but still it's embarrassing.
OS: MacOS X → All
Hardware: Macintosh → All
This bug exists.  I've seen it in Thunderbird 1.0.7, 1.5.0.2, and 1.5.0.4.  It's quite bothersome when there is a deep hierarchy of shared folders with most of the hierarchy marked \Noselect.

The user needs to click through errors for minutes on end for each startup.
I want to re-produce this bug. But can anyone tell me how to create a shared mailbox? Thanks
Re #4:

That depends on the server you're using.  With Cyrus IMAP you specify the paths in imapd.conf, like this:
sharedprefix: Shared Folders
Then you create top-level mailboxes using cyradm:
blabla> cm sharedtest
Set the ACL so anyone can see it:
blabla> sam sharedtest anyone read
blabla> lam sharedtest
anyone lrs

That's it.
Attachment #226773 - Flags: review?(bienvenu)
Comment on attachment 226773 [details] [diff] [review]
Don't issue "myrights" command on "Noselect" folders

thx, Boying
Attachment #226773 - Flags: superreview+
Attachment #226773 - Flags: review?(bienvenu)
Attachment #226773 - Flags: review+
So, shouldn't this bug be set to CONFIRMED or FIXED, then?
confirmed, I guess - afaik, the fix hasn't been checked in. Boying, do you want me to check the fix in?
Status: UNCONFIRMED → NEW
Ever confirmed: true
(In reply to comment #11)
> confirmed, I guess - afaik, the fix hasn't been checked in. Boying, do you want
> me to check the fix in?
> 

Yes. please check it in.  Thanks
David, can we put the patch into thunderbird2?
Attachment #226773 - Flags: approval-thunderbird2?
(In reply to comment #13)
> David, can we put the patch into thunderbird2?
> 

I mean can we put the patch into 1.8 branch?  Thanks
fixed on trunk and branch.
Status: NEW → RESOLVED
Closed: 13 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Attachment #226773 - Flags: approval-thunderbird2?
Flags: blocking1.8.0.6?
Flags: blocking1.8.0.7? → blocking1.8.0.7+
Comment on attachment 226773 [details] [diff] [review]
Don't issue "myrights" command on "Noselect" folders

approved for 1.8.0 branch, a=dveditz for drivers
Attachment #226773 - Flags: approval1.8.0.7+
fixed for 1.5.0.7
Keywords: fixed1.8.0.7
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.