[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 10:05) 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

Hm, looks like that after your last commit now. The server is still  
unhappy though:

(1297234921.0173) C: 2 SETANNOTATION INBOX "/comment" ("value.shared" {2+}
(1297234921.0177) C: Ä
(1297234921.0180) C: )
(1297234921.3045) S: 2 BAD Missing close paren in annotation  
attribute-values list


>>
>> 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.
>