[imp] Imp LDAP Error: [listMailboxes] IMAP error reported by server

[Andy Dorman]

On 02/09/2015 04:30 PM, Andy Dorman wrote:
> On 02/09/2015 01:43 PM, Michael M Slusarz wrote:
>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>
>>> Quoting Andy Dorman <adorman at ironicdesign.com>:
>>>
>>>> On 02/09/2015 10:18 AM, Michael J Rubinsky wrote:
>>>>>
>>>>> Quoting Jan Schneider <jan at horde.org>:
>>>>>
>>>>>> Zitat von Andy Dorman <adorman at ironicdesign.com>:
>>>>>>
>>>>>>> On 02/09/2015 08:26 AM, Andy Dorman wrote:
>>>>>>>> C: 6 LIST () "" (INBOX * "Other Users/*""" "Shared Folders/*""")
>>>>>>>> RETURN
>>>>>>>> (SUBSCRIBED)
>>>>>>>> S: 6 BAD Invalid syntax in List command
>>>>>>>
>>>>>>> I have found the problem...not sure of the fix yet.
>>>>>>>
>>>>>>> The problem is that the LIST command needs a space between the
>>>>>>> mailbox name and the reference name "pattern".
>>>>>>>
>>>>>>> For example, this LIST command works fine while the original above
>>>>>>> does not and the only difference is the space after the mailbox
>>>>>>> name:
>>>>>>>
>>>>>>> LIST () "" (INBOX * "Other Users/*" "" "Shared Folders/*" "") RETURN
>>>>>>> (SUBSCRIBED)
>>>>>>>
>>>>>>> I have not yet looked at the code where the command is being
>>>>>>> assembled.
>>>>>>>
>>>>>>> --
>>>>>>> Andy Dorman
>>>>>>>
>>>>>>> --
>>>>>>> imp mailing list
>>>>>>> Frequently Asked Questions: http://wiki.horde.org/FAQ
>>>>>>> To unsubscribe, mail: imp-unsubscribe at lists.horde.org
>>>>>>
>>>>>> I'm pretty sure this has once been reported elsewhere (ticket tracker
>>>>>> or mailing list) but couldn't be reproduced. Please search for the
>>>>>> original report.
>>>>>>
>>>>>
>>>>> Maybe https://bugs.horde.org/ticket/13825
>>>>>
>>>>
>>>> Right, I remember seeing the original issue but did not follow up as
>>>> it involved active sync.
>>>>
>>>> However, in reading the bug report it looks like the same issue.  The
>>>> addition of the empty reference name pattern "" without the
>>>> delimiting space between it and the mailbox name appears to be the
>>>> problem.
>>>>
>>>> I will add my debug log output and the results of manual testing with
>>>> Cyrus to the bug report.
>>>
>>> My shot-in-the-dark guess is that this PHP fix is broken (would
>>> explain why 5.5.21 and 5.6.5 break things):
>>>
>>> https://bugs.php.net/bug.php?id=68532
>>>
>>> But I'm out of country and won't be able to look at this anytime
>>> soon.  The best course of action would be for someone to come up with
>>> a reproducible test case to verify.
>>
>> Almost positive this is the regression that Jan reported:
>>
>> https://bugs.php.net/bug.php?id=68948
>>
>> People should make noise on that ticket to get this fixed.
>>
>> michael
>>
>> ___________________________________
>> Michael Slusarz [slusarz at horde.org]
>>
>
> I voted for it...did not know enough to make an intelligent
> comment...but I voted for it and sent the link to several users of our
> webmail system to vote for it.
>
> Hopefully others on this list will as well since it is a show stopper
> for Horde Imp with the latest PHP 5.5.21 & 5.6.5.
>

FWIW, I finally figured out that the issue with the IMAP LIST command 
was NOT that there was a missing space between the mailbox name and the 
empty reference name pattern.  The problem is the empty reference name 
pattern should not be there at all.

I realize this is almost certainly caused by the regression bug 
https://bugs.php.net/bug.php?id=68948

In the meantime I had made a very ugly and possibly very stupid 
temporary fix so our users can read their email.  I modified 
Horde/Imap/Client/Data/Format/Filter/Quote.php to NOT do anything (ie, 
do not add the "") if the length of the incoming bucket data is 0.

I only did this as a last resort and I am uncertain about potential 
side-effects.  I will post more as we learn more in using my ugly hack.

-- 
Andy Dorman