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

* Your Email Address
* Spam protection	Enter the letters below: .__ . .. ..__ .__ \| \\| \|\|__\|\| \\| \ \|__/\|/\\|\| \|\|__/\|__/
Comment	>>>> 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. >>
Attachment
Watch this ticket

Saved Queries