Tickets :: [#9895] deleteMailbox doesn't work

6.0.0-RC7

6/18/26

History
Attachments
Comment
Watch
Download

Summary	deleteMailbox doesn't work
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org, slusarz (at) horde (dot) org
Requester	vilius (at) lnk (dot) lt
Created	4/13/11 (5545 days ago)
Due
Updated	4/19/11 (5539 days ago)
Assigned	4/13/11 (5545 days ago)
Resolved	4/19/11 (5539 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

5712	Michael Slusarz	State ⇒ Resolved

5212	Git Commit	Comment #12	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9895~~: Replace 'unixhier' parameter with 'userhierarchy' for Cyrsql auth driver 2 files changed, 8 insertions(+), 4 deletions(-) http://git.horde.org/horde-git/-/commit/53d7867eef3cb3a85aff19aa9957cee5ac8954e1

4812	Git Commit	Comment #11	Reply to this comment
Changes have been made in Git for this ticket: Fix default userhierarchy for Cyrsql driver (~~Bug #9895~~). 2 files changed, 10 insertions(+), 22 deletions(-) http://git.horde.org/horde-git/-/commit/94c2dbc93c077f04a1c5dcf34c689689b09b40af

87	vilius (at) lnk (dot) lt	Comment #10 New Attachment: userhier.patch	Reply to this comment
The IMAP auth driver defaults to 'user.' So this should be fine there - I don't see what the issue could be with the IMAP driver. True, it seems that I've tested IMAP driver on the wrong server. And with this patch CyrSQL driver works too.

557	Jan Schneider	Comment #9	Reply to this comment
Theoretically this could be even something completely different than "user NS-separator", so the prefix must be specified as a configuration option. It should default to "user." though.

27	vilius (at) lnk (dot) lt	Comment #8	Reply to this comment
[Show Quoted Text - 9 lines][Hide Quoted Text] My users are under name space "user." I will try to harcode that into deleteMailbox arguments later today. I can confirm that adding "user." in front of the $mailbox works and mailbox gets deleted. Except that cyrsql expects the userhierarchy to be 'user/', not 'user.'. I have no idea which is correct, but it was this way in H3 also. Jan - can you provide input here? No. It depends on your imapd.conf configuration. From Cyrus manual: "unixhierarchysep: 0 Use the UNIX separator character ?/? for delimiting levels of mailbox hierarchy. The default is to use the netnews separator character ?.?." Mine is the default.

497	Michael Slusarz	Comment #7	Reply to this comment
The IMAP auth driver defaults to 'user.' So this should be fine there - I don't see what the issue could be with the IMAP driver.

207	Michael Slusarz	Comment #6 Assigned to Jan Schneider	Reply to this comment
My users are under name space "user." I will try to harcode that into deleteMailbox arguments later today. I can confirm that adding "user." in front of the $mailbox works and mailbox gets deleted. Except that cyrsql expects the userhierarchy to be 'user/', not 'user.'. I have no idea which is correct, but it was this way in H3 also. Jan - can you provide input here?

26	vilius (at) lnk (dot) lt	Comment #5	Reply to this comment
My users are under name space "user." I will try to harcode that into deleteMailbox arguments later today. I can confirm that adding "user." in front of the $mailbox works and mailbox gets deleted.

514	vilius (at) lnk (dot) lt	Comment #4	Reply to this comment
Sorry for not mentioning it, but I saw that $mailbox was never defined and did exactly what your patch does. Still there is a problem with userhierarchy parameter which is never defined in my case. Also I tried removing that parameter from $mailbox = line to no avail. My users are under name space "user." I will try to harcode that into deleteMailbox arguments later today.

418	Michael Slusarz	Comment #3 Assigned to Michael Slusarz State ⇒ Feedback	Reply to this comment
Cyrsql was definitely broken because $mailbox was never defined. But IMAP looked like it should work. Can you try this patch and report back?

98	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9895~~: Fix deleteMailbox for Cyrsql and Imap auth drivers 2 files changed, 9 insertions(+), 3 deletions(-) http://git.horde.org/horde-git/-/commit/7488ec8ac68e21317e92fde638f340fc1f9233b1

552	vilius (at) lnk (dot) lt	Comment #1 Priority ⇒ 2. Medium State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Horde Framework Packages Summary ⇒ deleteMailbox doesn't work Type ⇒ Bug	Reply to this comment
When trying to delete a user from Administration -> Users interface using CyrSQL or IMAP drivers I get "IMAP error: Mailbox does not exist". Just a guess, but this is probably becouse of in the past IMAP libraries used PHP imap_deletemailbox function, but now it uses direct socket connection and mailbox need to be specified with namespace? Like user.mailbox.