[Tickets #1733] NEW: imp/lib/api.php incorrect function _imp_userList()

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=1733
-----------------------------------------------------------------------
 Ticket             | 1733
 Created By         | jeffrey at crawford.homeunix.net
 Summary            | imp/lib/api.php incorrect function _imp_userList()
 Queue              | IMP
 Version            | 4.0.2
 State              | Unconfirmed
 Priority           | 2. Medium
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


jeffrey at crawford.homeunix.net (2005-04-08 13:49) wrote:

The Function _imp_userList() did not have the "Secret::read" function
translating the "admin_password" parameter: The function should look like:

function _imp_userList()
{
    $params = $_SESSION['imp']['admin']['params'];
    $params['admin_user'] = $params['login'];
    $params['admin_password'] = Secret::read(Secret::getKey('imp'),
$params['password']);
    require_once 'Horde/IMAP/Admin.php';
    $imap = &new IMAP_Admin($params);
    return $imap->listMailboxes();
}

This is how the function looked before I fixed it:
function _imp_userList()
{
    $params = $_SESSION['imp']['admin']['params'];
    $params['admin_user'] = $params['login'];
    $params['admin_password'] = $params['password'];
    require_once 'Horde/IMAP/Admin.php';
    $imap = &new IMAP_Admin($params);
    return $imap->listMailboxes();
}

This prevented the Users and Group admin functions from working because the
cyrus imapd account was being passed the wrong password