Tickets :: Comment on [#10892] Password not change in memory

* Your Email Address
* Spam protection	Enter the letters below: . ..___. .. . \| \|[__ \| \|\| \| \|__\|\| \|__\|\|___\__\|
Comment	> Hi, > > we are havign problems with password changing - is it correctly > changes on IMAP server but not in memory in Horde, so session is > broken and user must relogin (with new password). This is, maybe, > happening cos we are using memcached for sessions inside PHP. > > I started to digging deeper into this and i found out something > interesting. When password is changed it is reseted in cache via > resetCredentials function. This function is reseting password stored > in memory (in session). Problem is that password from session is NEVER > used for logging into IMAP server. > > Login to IMAP is done via Horde_Imap_Client which takes login and > password as argument (in $params) and these values are set ONLY in > constructor (this is done by Horde/IMP in createImapObject function > called in Auth.php). Problem is that IMAP object is initialized only > ONCE per IMAP login in our installation - so, when password changes, > it is correctly updated in session but gets NEVER updated in IMAP > object. I was doing some debug logging inside Horde and Horde/IMP and > it is really like this, constructor for Horde_Imap_Client is called > only once per login, the IMAP object is probably serialized and cached > somewhere. > > Sending patch which works for me: > > File: passwd/lib/Passwd.php > Function (at the end of file): resetCredentials > Code (i added 3 lines which starts by '$imp_imap'): > > static public function resetCredentials($old_password, $new_password) > { > if ($GLOBALS['registry']->getAuthCredential('password') == > $old_password) { > $GLOBALS['registry']->setAuthCredential('password', > $new_password); > > $imp_imap = > $GLOBALS['injector']->getInstance('IMP_Factory_Imap')->create(); > $imp_imap->ob = NULL; > > $imp_imap->createImapObject($GLOBALS['registry']->getAuth(), > $new_password, IMP_Auth::getAutoLoginServer()); > } > }
Attachment
Watch this ticket

Saved Queries