IMAP: Ignore empty ENABLED responses when switching mailboxes
commit8152670c3715769753ceec5ac55a04853b086450
authorJan Kundrát <jkt@kde.org>
Fri, 17 Jul 2015 17:47:04 +0000 (17 19:47 +0200)
committerJan Kundrát <jkt@kde.org>
Sat, 18 Jul 2015 08:59:52 +0000 (18 10:59 +0200)
treeafcfc8979e34fa9e794e48ddf65d1ed5c4e4043b
parentf7fd033d77ee85b9d59af85c7b89ef864abc177f
IMAP: Ignore empty ENABLED responses when switching mailboxes

Some versions of Cyrus send out extra ENABLED responses. Trojita already
contains code (see bug 329204) for filtering them out in case they look
relevant to CONDSTORE or QRESYNC, but it turns out that sometimes we get
back an empty ENABLED as well.

There is a couple of options here, either to play the guessing game and
add a kludge to silently eat the empty ENABLED to fix this particular
bug, or to change the code to accept any possible ENABLED response while
switching mailboxes.

BUG: 350006
CCBUG: 329204
Upstream-Bug: https://bugzilla.cyrusimap.org/show_bug.cgi?id=3898
Change-Id: I7ce7e663108d5b64a46580fb4aaa57c4cf379445
src/Imap/Tasks/ObtainSynchronizedMailboxTask.cpp
tests/Imap/test_Imap_Tasks_ObtainSynchronizedMailbox.cpp
tests/Imap/test_Imap_Tasks_ObtainSynchronizedMailbox.h