[Tickets #13153] Re: Bugs in the current implementation for PARTIAL limiting (RFC 5267 [4.4]) in Horde's Imap_Client library

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/13153
------------------------------------------------------------------------------
  Ticket             | 13153
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Bugs in the current implementation for PARTIAL limiting
                     | (RFC 5267 [4.4]) in Horde's Imap_Client library
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2014-05-02 23:20) wrote:

> Now what you are seeing is probably the following: internally,  
> Dovecot's SEARCH and SORT code is the same.  So limiting via PARTIAL  
> will work on SORT simply because the code being used is identical  
> and limitations on PARTIAL with SORT aren't being enforced.

Yup.  Exactly as I said.

src/imap/cmd-search.c and src/imap/cmd-sort.c both call  
cmd_search_parse_return_if_found

cmd_search_parse_return_if_found (src/imap/imap-search.c) calls  
search_parse_return_options (same file).  Which is where the PARTIAL  
return modified is parsed.

Looking at the code, it looks like CONTEXT and UPDATE are also both  
available... and sure enough, they are both accepted without error.   
And it looks like UPDATE is working for SORT also.  Hmmmmm...

I might have to ping Timo to see if he simply forgot to add the  
capability flag to the available list.  Or if there are issues with  
the code that I am not aware of.