[imapfilter-devel] Small patch for Mailbox._send_query()

[Hans-Werner Hilse]

Hi,

the attached patch is needed for >=imapfilter-2.0.3 in order to work
properly -- at least with the MS Exchange and the Cyrus IMAP server.

The problem is triggered by the change in mailbox.lua: Calling
Mailbox._send_query with no parameters (or nil, for that matter) will
issue an IMAP call "UID SEARCH ALL ", i.e. it will have a trailing
space. The MS Exchange server and the Cyrus IMAP server will issue an
error message. I'm not sure about what the IMAP standard really says
about that, but I think in order to have maximum compatibility, the
patch should go upstream. In my case, it was triggered by calling
Mailbox.select_all(). I'm not sure if at the moment select_all() works
for any IMAP server at all.

The issue is more complicated, though. I'd rather prefer if imapfilter
would error out when the IMAP server returns error messages. Maybe,
with some spare time, I'll get to give it a try...

-hwh
-------------- next part --------------
A non-text attachment was scrubbed...
Name: imapfilter-2.0.5-select_all.patch
Type: text/x-patch
Size: 733 bytes
Desc: not available
URL: <http://lists.hellug.gr/pipermail/imapfilter-devel/attachments/20071005/c7821917/attachment.bin>