Tickets :: [#5569] IMP-imap authentication broken

6.0.0-beta1

9/24/25

Summary	IMP-imap authentication broken
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	3. High
Owners	chuck (at) horde (dot) org
Requester	adrieder (at) sbox (dot) tugraz (dot) at
Created	07/19/2007 (6642 days ago)
Due	07/19/2007 (6642 days ago)
Updated	07/23/2007 (6638 days ago)
Assigned	07/19/2007 (6642 days ago)
Resolved	07/23/2007 (6638 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

07/23/2007 02:23:25 PM	Chuck Hagenbuch	State ⇒ Resolved

07/23/2007 08:41:23 AM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #10	Reply to this comment
Please try these commits: http://lists.horde.org/archives/cvs/Week-of-Mon-20070716/069474.html Much better :-) it seems to be working now...

07/20/2007 09:29:08 PM	Chuck Hagenbuch	Comment #9	Reply to this comment
Please try these commits: http://lists.horde.org/archives/cvs/Week-of-Mon-20070716/069474.html

07/19/2007 06:58:05 PM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #8	Reply to this comment
forgot... imp/config/conf.php: $conf['server']['server_list'] = 'shown';

07/19/2007 06:55:03 PM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #7	Reply to this comment
Does anyone else see anything like this? What setup _exactly_ do you have for hooks, realms, etc.? here the relevant (I think) settnings: horde/config/conf.php $conf['auth']['params']['app'] = 'imp'; $conf['auth']['driver'] = 'application'; horde/config/prefs.php $_prefs['fullname'] = array( 'value' => '', 'locked' => false, 'shared' => true, 'type' => 'text', 'hook' => true, 'desc' => _("Your full name:") ); $_prefs['from_addr'] = array( 'value' => '', 'locked' => false, 'shared' => true, 'type' => 'text', 'hook' => true, 'desc' => _("Your From: address:") ); The hooks looks for the fullname and from-address in an LDAP-directory using Auth::getBareAuth() if setm otherwise they return ''. imp/config/servers.php: There is a long list of imap and pop servers, each having a different realm, e.g. $servers['sbox'] = array( 'name' => 'sbox / student / Studierende', 'server' => 'xxx.xxx.xxx.xxx', 'hordeauth' => false, 'protocol' => 'imap/notls', 'port' => 143, 'maildomain' => 'sbox.tugraz.at', 'smtphost' => 'mailrelay.tugraz.at', 'smtpport' => 25, 'realm' => 'sbox', 'preferred' => '', 'admin' => array( 'params' => array( 'login' => 'imap', 'password' => 'secret', 'userhierarchy' => 'user.', 'protocol' => 'imap/notls', 'hostspec' => 'xxx.xxx.xxx.xxx', 'port' => 143 ) ), 'quota' => array( 'driver' => 'imap', 'params' => array(), ), 'acl' => array( 'driver' => 'rfc2086', ), ); The pref_uid is then saved as user@realm, eg. testuser@sbox I hope this helps...

07/19/2007 05:48:37 PM	Chuck Hagenbuch	Comment #6 State ⇒ Feedback	Reply to this comment
Does anyone else see anything like this? What setup _exactly_ do you have for hooks, realms, etc.?

07/19/2007 05:36:19 PM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #5	Reply to this comment
authentication works now, but now preferences are not loaded anymore :-(

07/19/2007 05:18:20 PM	Chuck Hagenbuch	Comment #4	Reply to this comment
Alright - that was old code, but I guess it never triggered before.

07/19/2007 05:18:02 PM	Chuck Hagenbuch	Deleted Original Message

07/19/2007 05:09:41 PM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #3	Reply to this comment
No, that patch would break plenty of other cases. Fixed in CVS. No not yet .... in imp/lib/Auth/imp.php on line 130 the IMP_IMAP::singleton call passes $userID which is set to $_SESSION['imp']['uniquser'], so it still doesn't work.

07/19/2007 04:20:10 PM	Chuck Hagenbuch	Comment #2 State ⇒ Resolved Assigned to Chuck Hagenbuch	Reply to this comment
No, that patch would break plenty of other cases. Fixed in CVS.

07/19/2007 09:11:18 AM	adrieder (at) sbox (dot) tugraz (dot) at	Comment #1 Priority ⇒ 3. High State ⇒ Unconfirmed New Attachment: IMAP.php.patch Queue ⇒ IMP Due ⇒ 07/19/2007 Summary ⇒ IMP-imap authentication broken Type ⇒ Bug	Reply to this comment
The following patch breaks the IMP-imap authentication when using realm from a server list: http://cvs.horde.org/diff.php/imp/lib/IMAP.php?r1=1.44&r2=1.45&ty=u the attached patch solves the problem.