[Tickets #6042] php error on login page and a load delay because of an unauthenticated IMAP-access

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=6042
-----------------------------------------------------------------------
 Ticket             | 6042
 Created By         | m.gabriel at das-netzwerkteam.de
 Summary            | php error on login page and a load delay because of an unauthenticated
                    | IMAP-access
 Queue              | Kolab
 Type               | Bug
 State              | Unconfirmed
 Priority           | 3. High
 Owners             | 
-----------------------------------------------------------------------


m.gabriel at das-netzwerkteam.de (2007-12-24 19:01) wrote:

i am using kolab-2.1 (debian unstable) and horde-webmail-1.1rc1

with the patch below an error on the login.php page vanishes (when using
IMAP auth with kolab). the error was called, if all IMAP folders of the
kolab-cyrus server are accessible only for authenticated IMAP users.

Warning: Invalid argument supplied for foreach() in
/usr/local/share/_horde-versions_/horde-webmail-1.1-rc1-netzwerkteam/lib/Horde/Kolab/IMAP.php
on line 357

there is still a delay in loading the login.php page, though, as nag tries
to access some IMAP-folders prior to authentication:

Dec 24 23:58:42 grimnir cyrus/imap[5860]: badlogin: localhost [127.0.0.1]
plaintext  SASL(-1): generic failure: All-whitespace username.
Dec 24 23:58:45 grimnir HORDE[15339]: [nag] Unable to authenticate with
the Kolab IMAP server [on line 1689 of
"/usr/local/share/_horde-versions_/horde-webmail-1.1-rc1-netzwerkteam/lib/Horde/Kolab/IMAP.php"]

here comes the patch...

diff -r horde-webmail-1.1-rc1/lib/Horde/Kolab/IMAP.php
horde-webmail-1.1-rc1-netzwerkteam/lib/Horde/Kolab/IMAP.php
356,360c356,362
<             foreach ($folders as $folder) {
<                 if ($folder[1] == $this->_object_type && $folder[2] ==
1) {
<                     return $folder[0];
<                 }
<             }
---
>           if ($folders) {
>                 foreach ($folders as $folder) {
>                           if ($folder[1] == $this->_object_type &&
$folder[2] == 1) {
>                       return $folder[0];
>                     }
>               }
>           }

mike