On Fri, 2009-02-20 at 17:48 +0100, Bernhard Herzog wrote:
> On 18.02.2009, Sascha Wilde wrote:
> > according to RfC4314 the rights argument to the setacl command might be
> > an empty string ("zero right characters"):
> >
> >     The third argument is a string containing an optional plus ("+") or
> >     minus ("-") prefix, 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.
> 
> Below is a patch that fixes this.  SETACL with an empty string as rights will 
> be equivalent to DELETEACL with the same identifier.

Committed.

> While testing this, I noticed two other problems in the imap-acl plugin, both 
> related to negative ACL entries.  If an ACL contains both negative and 
> positive entries for the same identifier, no space is output in the the 
> GETACL response, leading to something like e.g.
> 
> * ACL "INBOX/foo" "someuser" lrs-"someuser" w 

Fixed.

> The second problem is that the "-" is not inside the double quotes.

Fixed.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to