Re: strange getinfo result

Hi Marco,

Looks like it's just a concatenation bug.  The annotations are of the form "/some/key = value", and when preparing the the results, the function adds "/mailbox/" or "/server/" to the front of the key (depending on if it's a mailbox or server annotation) -- and so the "/" gets doubled up.  The correct behaviour would be to add "/mailbox" or "/server" without the extra slash.

It'll probably get fixed at some point (a PR would be great!).  But I don't have a good data set to test with, so I won't just quickly fix it now myself (in case I miss a detail and make it worse).  I've opened https://github.com/cyrusimap/cyrus-imapd/issues/2666 so it doesn't get forgotten :)

Cheers,

ellie

On Mon, Mar 11, 2019, at 9:41 PM, Marco wrote:
> Hello,
> 
>   just a minor detail, but I would like to know it, if it could change 
> in near future. If I call getinfo with Cyrus::IMAP::Admin (Cyrus-Imapd 
> 3.x) on a mailbox I see something like this:
> 
> (
>    "user/myname\@example.com",
>    {
>      private => {
>                   "/mailbox//check" => "NIL",
>                   "/mailbox//checkperiod" => "NIL",
>                   "/mailbox//comment" => "NIL",
>                   "/mailbox//sort" => "NIL",
>                   "/mailbox//specialuse" => "NIL",
>                   "/mailbox//thread" => "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/expire" => "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/news2mail" => "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/sieve" => "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/squat" => "NIL",
>                 },
>      shared  => {
>                   "/mailbox//check"                                   => 
> "NIL",
>                   "/mailbox//checkperiod"                             => 
> "NIL",
>                   "/mailbox//comment"                                 => 
> "NIL",
>                   "/mailbox//sort"                                    => 
> "NIL",
>                   "/mailbox//specialuse"                              => 
> "NIL",
>                   "/mailbox//thread"                                  => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/annotsize"        => 0,
>                   "/mailbox//vendor/cmu/cyrus-imapd/duplicatedeliver" => 
> "false",
>                   "/mailbox//vendor/cmu/cyrus-imapd/expire"           => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/lastpop"          => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/lastupdate"       => 
> "15-Feb-2019 11:55:03 +0100",
>                   "/mailbox//vendor/cmu/cyrus-imapd/news2mail"        => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/partition"        => 
> "maildatabig",
>                   "/mailbox//vendor/cmu/cyrus-imapd/pop3newuidl"      => 
> "true",
>                   "/mailbox//vendor/cmu/cyrus-imapd/pop3showafter"    => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/sharedseen"       => 
> "false",
>                   "/mailbox//vendor/cmu/cyrus-imapd/sieve"            => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/size"             => 
> 2562,
>                   "/mailbox//vendor/cmu/cyrus-imapd/squat"            => 
> "NIL",
>                   "/mailbox//vendor/cmu/cyrus-imapd/synccrcs"         => 
> "836105806 1993713954",
>                   "/mailbox//vendor/cmu/cyrus-imapd/uniqueid"         => 
> "2r1e4dff8u5h23f5qmqrqn77",
>                 },
>    },
> )
> 
> 
> My question is: why does the double slash ('//') after 'mailbox' appear?
> 
> Maybe did you think
> 
> 	"/mailbox/vendor/..."
> 
> or maybe
> 
> 	"/mailbox/[email protected]/vendor/..."
> 
> 
> Thank you very much
> Marco
> ----
> Cyrus Home Page: http://www.cyrusimap.org/
> List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
> To Unsubscribe:
> https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
>
----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
To Unsubscribe:
https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus