[Tickets #9565] Re: Imap_Client fails with literal data on setMetadata command

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9565
------------------------------------------------------------------------------
  Ticket             | 9565
  Updated By         | Gunnar Wrobel <wrobel at pardus.de>
  Summary            | Imap_Client fails with literal data on setMetadata
                     | command
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Gunnar Wrobel <wrobel at pardus.de> (2011-02-09 02:39) wrote:

>> Working output would be:
>>
>> DEBUG: C: A0002 SETANNOTATION INBOX "/comment" ("value.shared" {2+}
>> DEBUG: C: Ä)
>> DEBUG: S: A0002 OK Completed
>
> This is incorrect also.  This should be:
>
> DEBUG: C: A0002 SETANNOTATION INBOX "/comment" ("value.shared" {2+}
> DEBUG: C: Ä
> DEBUG: C: )
> DEBUG: S: A0002 OK Completed
>
> The ')' is part of the IMAP command, not the literal, so it needs to  
> appear on a new line.

Ah, okay. The above command I posted was generated by the roundcube  
IMAP driver and I just copied it. The server apparently did not care  
about the misplaced parenthesis.

>
> This has been fixed (I think; don't have an easy way to test at the moment):

It fails with

(1297208022.0143) C: 2 SETANNOTATION
(1297208022.1472) S: 2 BAD Missing required argument to Setannotation

now. Testing is easy on my side as this just requires a short command  
line call. If you need more info just tell me. Thanks for  
investigating this!

> http://lists.horde.org/archives/commits/2011-February/006677.html
>
> It also looked like the 'SETMETADATA' method was also broken - this  
> should be fixed also.