Re: Patch: add new lmtptarget annotation

2010-05-20 Thread Kevin Kobb
On 5/18/2010 12:38 PM, Stephen Grier wrote: All, Just submitting a patch I'm supporting locally for consideration. We use shared mailboxes quite extensively for role-based communication. For quite some time we've had a problem with users deleting or renaming mailboxes into which we deliver

Re: Patch: add new lmtptarget annotation

2010-05-20 Thread Wesley Craig
On 18 May 2010, at 15:11, Nic Bernstein wrote: Where in the documentation should information be added describing this new annotation? Where are available annotations documented currently? As far as I can tell, there this:

Patch: add new lmtptarget annotation

2010-05-18 Thread Stephen Grier
All, Just submitting a patch I'm supporting locally for consideration. We use shared mailboxes quite extensively for role-based communication. For quite some time we've had a problem with users deleting or renaming mailboxes into which we deliver mail. We can, and do, use IMAP ACLs to dissallow

Re: Patch: add new lmtptarget annotation

2010-05-18 Thread Wesley Craig
Seems like a reasonable approach and a good patch. I might suggest some feedback, preferably to the deleting / renaming user but a syslog might also do, when the delete / rename failed. Is there a bugzilla number? :wes On 18 May 2010, at 12:38, Stephen Grier wrote: Just submitting a patch

Re: Patch: add new lmtptarget annotation

2010-05-18 Thread Frank Richter
Hello Stephen, We use shared mailboxes quite extensively for role-based communication. For quite some time we've had a problem with users deleting or renaming mailboxes into which we deliver mail. We can, and do, use IMAP ACLs to dissallow users from deleting the delivery target mailbox. But

Re: Patch: add new lmtptarget annotation

2010-05-18 Thread Nic Bernstein
This looks like a great addition. One question: Where in the documentation should information be added describing this new annotation? Where are available annotations documented currently? Thanks for the contribution! -nic On 05/18/2010 12:47 PM, Wesley Craig wrote: Seems like a

Re: Patch: add new lmtptarget annotation

2010-05-18 Thread Stephen Grier
On 18/05/10 18:47, Wesley Craig wrote: Seems like a reasonable approach and a good patch. I might suggest some feedback, preferably to the deleting / renaming user but a syslog might also do, when the delete / rename failed. My patch returns IMAP_MAILBOX_NOTSUPPORTED to the client. I can

Re: Patch: add new lmtptarget annotation

2010-05-18 Thread Adam Tauno Williams
On Tue, 2010-05-18 at 20:53 +0200, Frank Richter wrote: Hello Stephen, We use shared mailboxes quite extensively for role-based communication. For quite some time we've had a problem with users deleting or renaming mailboxes into which we deliver mail. We can, and do, use IMAP ACLs to