6.0.0-beta6
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
3/31/26
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#12097] Export of mailfolders fails with cyrus imap
*
Your Email Address
*
Spam protection
Enter the letters below:
__ .__..___. .. . / `| | _/ || | \__.|__\./__.\__||__|
Comment
>>> The problem is in the method "update" in .../Imap/Client/Ids/Map.php >>> which is not checking whether the mapping array contains only valid >>> values (I guess they should always be int?). >> >> Yes. So the question is... WHY are they not all integers for you? > I doubt this is only a problem of our setup because looking in my > IMAP logs I can't find any non rfc answer on the server side which > could explain the problem. > >> Your patch is not useful because it masks the problem but doesn't fix it. > > You are right it doesn't fix the problem - I traced the bug and found > a problem in the method "_updateCache" in class > "Horde_Imap_Client_Base". The mapping array is created unchecked > using the getUid() which is either an integer or "null". So I would > suggest to check the retrieved uid before assigning it to the mapping > array. > > Again I attached a patch. > > You may say: This is also not solving the origin problem but I think > as long as you cannot be sure the method returns integer values in > all cases you have to check the result in calling processes. > (Btw: It is not nescessary to update the cached mapping anyway to > export mails successfully.) > >
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers