6.0.0-beta1
7/7/25

[#11440] Imap Fetch fails with invalid UID
Summary Imap Fetch fails with invalid UID
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Not A Bug
Priority 2. Medium
Owners
Requester horde (at) gosign (dot) de
Created 09/23/2012 (4670 days ago)
Due
Updated 10/15/2012 (4648 days ago)
Assigned
Resolved 10/10/2012 (4653 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/15/2012 08:39:46 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (develop):

commit 72d5968e097b1541bbfcdc627adf2ef5999fb273
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Sun Sep 23 21:46:12 2012 -0600

     Bug #11440: Fix VANISHED FETCH search

  .../Imap_Client/lib/Horde/Imap/Client/Socket.php   |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/72d5968e097b1541bbfcdc627adf2ef5999fb273
10/10/2012 06:22:22 AM Michael Slusarz Comment #2
State ⇒ Not A Bug
Reply to this comment
This code doesn't exist in Git.  At best, it was fixed during the IMAP 
parsing rewrite.
09/23/2012 10:04:21 PM horde (at) gosign (dot) de Comment #1
Priority ⇒ 2. Medium
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Imap Fetch fails with invalid UID
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
Horde Imap does not seem to adhere to RFC3501 and therefore fails on 
Cyrus as it does strict argument checking.

Discussed here: 
http://www.dovecot.org/list/dovecot/2012-April/065086.html already 
some parts fixed in bug #11132.
There is however another point in Imap/Client/Socket.php  where this 
needs to be fixed:
in line 3167 the empty parameter is
hard-coded:

         $this->_sendLine(array(
              'UID FETCH',
              $ids->all ? '1:*' : strval($ids),
              array(),
              array(
                  'VANISHED',
                  'CHANGEDSINCE',
                  array('t' => Horde_Imap_Client::DATA_NUMBER, 'v' =>
intval($modseq))
              )


this can be fixed by changing it to:

   $this->_sendLine(array(
              'UID FETCH',
              $ids->all ? '1:*' : strval($ids),
              array('UID'),
              array(
                  'VANISHED',
                  'CHANGEDSINCE',
                  array('t' => Horde_Imap_Client::DATA_NUMBER, 'v' =>
intval($modseq))
              )


I also commented on the other bug metioned above, but could not figure 
out how to reopen it. sorry.


Alex

Saved Queries