Tickets :: Comment on [#11924] Add API in Horde_Registry

* Your Email Address
* Spam protection	Enter the letters below: . .. ..__.\ /.__ \|\ \|\| \|\| \| >< [__) \| \\|\|/\\|\|__\/ \\|
Comment	>> I like the idea of just resetting auth and forcing the app to reauth though. >> I doubt this will work, at least for IMP. > > That's exactly what I proposed. So I am confused. > > Note that you can't separate the "authentication" of an application > from its session data. They are tied together. In other words: in > IMP you can't expect changing the password in the IMP object is all > that is needed. There may be other session data (i.e. data added to > the session by the user via configuration/hooks) that are tied to > that previous password. So it's all or nothing when clearing an > application. > >> After the password is changed in the backend, I cannot call clearAuth >> or clearAuthApp because it would run pushApp, >> IMP_Application::_authenticated and in turn >> IMP_Auth::authenticateCallback. This would use the old invalid >> credentials and result in the dreaded "IMP NOT ACTIVATED" message. > > I don't understand. If you call clearAuth(), it will attempt to call > IMP's 'logout' method as you described. If it fails (which it will > in this situation), this exception should be caught and ignored > within clearAuth(). > > If you call clearAuthApp(), the calling code should be responsible > for catching and ignoring the exception.
Attachment
Watch this ticket