Re: [Dovecot] v1.2: can't access other users shared INBOX

2009-03-30 Thread Bernhard Herzog
On 25.03.2009, Bernhard Herzog wrote: > On 19.03.2009, Bernhard Herzog wrote: > > The reason for that is maildir_fill_readdir always adds the virtual name > > of the INBOX even when MAILBOX_LIST_ITER_VIRTUAL_NAMES isn't set. In > > lines 260ff of mailbox-list-maildir-ite

Re: [Dovecot] v1.2: can't access other users shared INBOX

2009-03-25 Thread Bernhard Herzog
On 19.03.2009, Bernhard Herzog wrote: > The reason for that is maildir_fill_readdir always adds the virtual name of > the INBOX even when MAILBOX_LIST_ITER_VIRTUAL_NAMES isn't set. In lines > 260ff of mailbox-list-maildir-iter.c, rev. 5284f45c249a it unconditionally > adds the

Re: [Dovecot] v1.2: can't access other users shared INBOX

2009-03-19 Thread Bernhard Herzog
On 17.03.2009, Bernhard Herzog wrote: > That's A's INBOX, most likely, so it should be accessible. That it's > listed but not accessible is AFAICT a combination of two bugs. One is that > the INBOX's ACL is used as default, so if B as l-permission in A's INBOX

Re: [Dovecot] v1.2: can't access other users shared INBOX

2009-03-17 Thread Bernhard Herzog
/src/lib-storage/index/maildir/maildir-storage.c Tue Mar 17 16:38:57 2009 +0100 @@ -201,8 +201,7 @@ maildir_create(struct mail_storage *_sto list_set.lock_method = &_storage->lock_method; if (list_set.inbox_path == NULL && - strcmp(layout, MAILDIR_PLUS

Re: [Dovecot] ACLs are applied recursively to sub mailboxes

2009-03-17 Thread Bernhard Herzog
On 16.03.2009, Timo Sirainen wrote: > On Mon, 2009-03-16 at 20:33 +0100, Bernhard Herzog wrote: > > That pathological aclobj is created in acl_backend_init: > > backend->default_aclobj = acl_object_init_from_name(backend, NULL, ""). > &g

Re: [Dovecot] ACLs are applied recursively to sub mailboxes

2009-03-16 Thread Bernhard Herzog
On 13.03.2009, Bernhard Herzog wrote: > On 10.03.2009, Timo Sirainen wrote: > > I've been a bit busy (or lazy) recently and I'm not focusing on trying > > to get the new dbox code working. I'll look at the ACL bugs at some > > point, but you can probably get the

Re: [Dovecot] ACLs are applied recursively to sub mailboxes

2009-03-13 Thread Bernhard Herzog
look into this. Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner signature.asc Description: This is a digitally signed message part.

Re: [Dovecot] v1.2 can't set ACL to empty string

2009-02-20 Thread Bernhard Herzog
x, followed by zero or more rights characters. > > existing clients (horde in particular) actually use this to remove all > rights from an user. > > Currently dovecot 1.2 does not accept an empty rights string as argument > to setacl. Bernhard Herzog will look into this. Be

Re: [Dovecot] [patch] 'c' and 'd' in setacl

2009-02-09 Thread Bernhard Herzog
g/265967 Bernhard [1] ANSI C89 draft: http://flash-gordon.me.uk/ansi.c.txt linked to from http://en.wikipedia.org/wiki/ANSI_C -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koor

Re: [Dovecot] IMAP ACLs and global ACLs in v1.2

2009-02-05 Thread Bernhard Herzog
"owner" ACL entries so that we can get on with testing. I don't think the patch is the right solution to the problem, but I've included it anyway. Maybe it's useful for a discussion. Cheers, Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.in

Re: [Dovecot] patch: list shared namespace

2008-11-04 Thread Bernhard Herzog
On 31.10.2008, Timo Sirainen wrote: > On Tue, 2008-10-28 at 21:07 +0100, Bernhard Herzog wrote: > > I'm not sure the new hook is really needed. The patch could perhaps > > just as well extend the acl_next_hook_mail_storage_created and > > acl_next_hook_mailbox_list_c

Re: [Dovecot] patch: list shared namespace

2008-11-04 Thread Bernhard Herzog
orary developer debugging thing, or > will it be useful also for sysadmins? I put that in because it was useful for me while developing. I'm not sure it's useful for admins later on. So maybe I should remove them. Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | ht

Re: [Dovecot] patch: list shared namespace

2008-11-03 Thread Bernhard Herzog
regardless of whether foo, foo.foo or foo.bar actually exist. foo, foo.foo and foo.bar are added to the list as nonexisting mailboxes automatically, though. So AFAICT from the other user's point of view it really is as if only foo.foo.foo and foo.bar.baz actually existed. Of course, assuming there's a reason acl_mailbox_try_list_fast has a "try" in its name and that it actually can fail, foo, foo.foo and foo.bar could perhaps end up in the mailbox list even if they do not have children that are visible to the user. Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner signature.asc Description: This is a digitally signed message part.

Re: [Dovecot] patch: list shared namespace

2008-10-31 Thread Bernhard Herzog
o" as non-existing > LIST foo -> List "foo" as non-existing > LIST * -> List "foo/bar" only Maybe there are circumstances that I didn't encounter yet, where it does indeed fail. Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevat

Re: [Dovecot] patch: list shared namespace

2008-10-31 Thread Bernhard Herzog
On 28.10.2008, Bernhard Herzog wrote: > - The dovecot-acl-list is not always rebuilt, even when it should have >been, AFAICT. In particular, if the file exists but is empty, it's >never updated, even when ACL later change. Maybe this is a bug in >the Kolab branch.

Re: [Dovecot] imap dump-capability fails Dovecot 1.1.6

2008-10-31 Thread Bernhard Herzog
); } if (pipe(fd) < 0) { The Patch is not an attachment this time to avoid mailman breaking the openpgp signature. Regards, Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück,

Re: [Dovecot] patch: list shared namespace

2008-10-31 Thread Bernhard Herzog
On 28.10.2008, Bernhard Herzog wrote: > - List with "%" doesn't list all intermediate mailboxes. > >On the one hand arthur sees this: > > x LIST "" "*" > ... > * LIST (\Noselect \HasChildren) "/" "users/ford

Re: [Dovecot] patch: list shared namespace

2008-10-29 Thread Bernhard Herzog
On 28.10.2008, Bernhard Herzog wrote: > I've been working on a patch for dovecot 1.2 from the Kolab branch > (http://hg.intevation.org/kolab/dovecot-1.2_kolab-branch/) that > implements listing of shared namespaces. I've got something that works > in some basic way but

[Dovecot] patch: list shared namespace

2008-10-28 Thread Bernhard Herzog
x LIST "" "users/ford/%" * LIST (\Noselect \HasChildren) "/" "users/ford/INBOX" x OK List completed. At least Kontact resp. KMail rely on this. - The dovecot-acl-list is not always rebuilt, even when it should have been, AFAICT. In particular

Re: [Dovecot] imap segfaults in dovecot 1.2 on logout

2008-10-15 Thread Bernhard Herzog
On 15.10.2008, Bernhard Herzog wrote: > I recall having a similar problem with the Annotation plugin. IIRC it had > something to do with not unregistering commands properly when the plugin > was unloaded. In that case the array of known commands retains dangling > pointers to the

Re: [Dovecot] imap segfaults in dovecot 1.2 on logout

2008-10-15 Thread Bernhard Herzog
mmands that point to the unloaded shared object file. Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner signature

Re: [Dovecot] dovecot 1.2: dict no longer really s upport DICT_ITERATE_FLAG_RECURSE

2008-10-13 Thread Bernhard Herzog
ct order by clause. After that it worked fine. Regards, Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagne

Re: [Dovecot] dovecot 1.2: dict no longer really support DICT_ITERATE_FLAG_RECURSE

2008-10-09 Thread Bernhard Herzog
On 09.10.2008, Timo Sirainen wrote: > On Oct 9, 2008, at 6:15 PM, Bernhard Herzog wrote: > > This probably has to do with the pattern matching for dict keys in > > the config > > file. It would be enough for my purposes at the moment -- i.e. > > getting the >

[Dovecot] dovecot 1.2: dict no longer really support DICT_ITERATE_FLAG_RECURSE

2008-10-09 Thread Bernhard Herzog
my purposes at the moment -- i.e. getting the metadata plugin to work with dovecot 1.2 -- if recursion were only possible for the last of fields in the pattern. Regards, Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnab

Re: [Dovecot] Bug in dovecot 1.2 dict

2008-10-02 Thread Bernhard Herzog
ict_server_init. The attached patch fixes it for me. Regards, Bernhard -- Bernhard Herzog | ++49-541-335 08 30 | http://www.intevation.de/ Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998 Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner diff

Re: [Dovecot] Annotation plugin for dovecot

2008-09-05 Thread Bernhard Herzog
On 05.09.2008, Robert Schetterer wrote: > that the kolab server will switch from imap cyrus to imap dovecot someday? This came up on the kolab-devel mailing list recently too: http://kolab.org/pipermail/kolab-devel/2008-July/009314.html The upshot is that there are no concrete plans to actually r

[Dovecot] Annotation plugin for dovecot

2008-09-05 Thread Bernhard Herzog
Hi, as part of an effort to make dovecot usable as the IMAP component in the Kolab Groupware system , we've started to develop a plugin that implements the IMAP METADATA extension. The plugin is still under development and development is focused on the plugin's use in Kolab,