[Tickets #10228] Re: Add API methods to reset cached credentials inside applications

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/10228
------------------------------------------------------------------------------
  Ticket             | 10228
  Updated By         | Jan Schneider <jan at horde.org>
  Summary            | Add API methods to reset cached credentials inside
                     | applications
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Enhancement
-State              | Accepted
+State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2011-10-18 13:17) wrote:

>> Paste from Mailing list:
>>
>> I am using the passwd code from git which I downloaded on June 6. It
>> works, but does not reset my credentials within Horde, so I have to
>> log out and log back in. My system is setup such that Horde
>> authenticates via Imp and Imp uses IMAP Authentication. When I change
>> my password with passwd, it successfully changes the password but IMP
>> keeps trying to authenticate with the old password.
>
> Hi I'm using the passwd code too, and I got the same problem whit it  
> . I found that that the problem is  inside the function  
> resetCredentials on passwd/lib/Passwd.php, first because the call to  
> setAuthCredential() to set the new password is made on wrong way, we  
> have to change from:
>
> $GLOBALS['registry']->setAuthCredential('password', $new_password);
>
>  to:
>
> $GLOBALS['registry']->setAuthCredential('password', $new_password,  
> $GLOBALS['session']->get('horde', 'auth/credentials'));
>
>
> That's because the third parameter (The app name) on function  
> setAuthCredential is bad calculate on class Horde_Registry when it  
> is not passed to if function (because is optional), but the same  
> parameter  on function getAuthCredential is rigth calculated when  
> it's missing, so it could be a bug on  setAuthCredential.

I think it's a bug there, can you try what I just committed?