[Pkg-Cyrus-imapd-Debian-devel] Bug#347660: please look at the wildcard-deactivating patches (Kolab)

[Henrique de Moraes Holschuh]

On Wed, 11 Jan 2006, Steffen Joeris wrote:
> This is necessary because Kolab uses the whole email adress for the
> mailbox and there are for example '%' 's :)

I will not take this patch. It would be asking for misterious breakages.

Also, I am not sure it is legal IMAP either, and if it is not legal IMAP per
the RFCs, I'd push for its removal even if it were accepted...

> We know that it will be very difficult or maybe impossible to integrate it
> into the official cyrus package, but i think we will discuss it

Indeed.  Why not fix Kolab so that it stops doing such weird things? It
should be doable.

> +  #ifdef notdef
>      /* verify that the mailbox doesn't have a wildcard in it */
>      for (p = oldmailboxname; !r && *p; p++) {
>         if (*p == '*' || *p == '%') r = IMAP_MAILBOX_BADNAME;
>      }
> +  #endif

We *really* don't want to mess with this.

> + * original definition
>  #define GOODCHARS " +,-.0123456789:=@ABCDEFGHIJKLMNOPQRSTUVWXYZ_abcdefghijklmnopqrstuvwxyz~"
> + */
> +
> +#define GOODCHARS " #$%'()*+,-.0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[]^_`abcdefghijklmnopqrstuvwxyz{|}~"
> +

Some of those might be needed for other stuff... as a rule, cyrus avoids
chars that could cause hell on scripts or the imap protocol, and reserves
some itself. So no ^ (unixhiersep), no wildcards, no {, no }, etc.

Exactly what kind of crazy thing does kolab do with mailboxes that it needs
the above changes?

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh