[Tickets #8353] Re: Sort out LDAP preferences mess with user bindings

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/8353
------------------------------------------------------------------------------
  Ticket             | 8353
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | Sort out LDAP preferences mess with user bindings
  Queue              | Horde Base
  Version            | 3.3.4
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2009-06-16 12:58) wrote:


> remember the failure regardless of the user we are attempting to  
> connect as. So any further attempts at prefs access - even if from  
> the current user - are never even attempted.

Looking further, I misread the code in that method dealing with  
remembering the errror. We only check the session for a previous error  
if _connect returns an error - so this specific part isn't an issue in  
this case.  It would only be an issue in that it could hide a later,  
unrelated error...a very fringe case I would think.  I still think  
comparing Prefs::_user and Auth::getAuth() before pushing the  
notification would be a reasonable way to go here though.