mailbox deletion through sieve and cyrus does not work
mailbox deletion through sieve and cyrus does not work
Hello,
i have a cyrus server correctly configured that can create mailbox, that works well but the deletion doesnt work
i tried like this :
- create a new user
- add a mail account
- delete the user
the deletion from ldap works well, user is no longer here
the mailbox stays in the cyrus server
Cheers
(from redmine: issue id 3880, created on 2015-06-17, closed on 2016-01-06)
- Changesets:
- Revision 766563e2 by Côme Chilliet on 2015-11-03T05:29:15.000Z:
Fixes #3880 Mailbox deletion was not called anymore- Revision 9d61c48b by Côme Chilliet on 2015-11-03T05:30:41.000Z:
Fixes #3880 Mailbox deletion was not called anymore- Custom Fields:
- Bug in version: 1.0.8.7
Activity
hello,
more info given to me in a mailexchange
am running cyrus 2.4.16. Yes, by default the cyrus admin user cannot delete a mailbox, but it can grant itself permission to delete a mailbox, and then delete it. I have done this on my system at least. The deleteMailbox code for the cyrus plugin actually does this:
/* grant all permission to cyrus admin user / @imap_setacl ($this->imap_handle, $this->account_id, $cfg["admin"], "lrswipcda"); ... / then delete the mailbox */ imap_deletemailbox($this->imap_handle, this->account_id)
(from redmine: written on 2015-11-01)
Indeed the mailbox deletion method was not called… This bug is really old, ever since the mail tab uses simplePlugin…
I pushed a fix, but there is a problem: the mailbox deletion will get called even if the LDAP deletion of the tab fails. That seems problematic. The best way to fix this is to set post_remove method to get called only if ldap_remove method returns no errors. This means that postremove hooks won’t get called if the ldap remove fails. This seems like a good idea to me but I wanted to get your opinion first as it is a behavior change (and 1.0.9.2 is supposed to be a minor version).
(from redmine: written on 2015-11-03)
added Fixed label
