[Tickets #3687] imap_append in Message.php fails with no errors

bugs@bugs.horde.org bugs at bugs.horde.org
Sat Mar 25 10:40:54 PST 2006


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

Ticket URL: http://bugs.horde.org/ticket/?id=3687
-----------------------------------------------------------------------
 Ticket             | 3687
 Updated By         | david-shafer at uiowa.edu
 Summary            | imap_append in Message.php fails with no errors
 Queue              | IMP
 Version            | 4.1
 State              | Feedback
 Priority           | 2. Medium
 Type               | Bug
 Owners             | Michael Slusarz
-----------------------------------------------------------------------


david-shafer at uiowa.edu (2006-03-25 10:40) wrote:

Thanks for checking it out. I'm sure it works for many people without the
change, but it definitely wasn't working for us. Using tcpdump showed PHP
wasn't even attempting the append operation; it returned false immediately,
as though it didn't like the format of one of the parameters. It made sense
after I looked at other calls to imap_append(). Why wouldn't both calls to
imap_append() in Message.php use the same calling syntax (with a fully
qualified "{server}mailbox" rather than just "mailbox")? The sample in the
PHP documentation uses a fully qualified mailbox as well:

http://www.php.net/manual/en/function.imap-append.php

The way it is now, I'm sure it works for many sites, but it definitely
doesn't work for all. I'm not surprised more people haven't noticed it,
because this code probably isn't used often. It's only used when the trash
folder is enabled, and the user's quota prevents them from copying a deleted
message to the trash, and apparently it's only a problem for certain
configurations.

Fyi, on the web server we're using php 4.3.9 (the Red Hat Enterprise Linux 4
package php-4.3.9-3.9) with c-client 2002e (libc-client-2002e-14). The IMAP
server is Cyrus 2.2.12 (cyrus-imapd-2.2.12-3.RHEL4.1).




More information about the bugs mailing list