[dev] [commits] Horde branch master updated. f393c842d15ddf88d0b89ceaffc2b27a1e19595b

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> commit f393c842d15ddf88d0b89ceaffc2b27a1e19595b
>> Author: Jan Schneider <jan at horde.org>
>> Date:   Thu May 27 16:26:52 2010 +0200
>>
>>    Unless I'm missing something, we don't want to require Horde  
>> authentication on
>>    the login screen. If we do, and authentication fails, e.g. because we are
>>    redirected here from some other page because of session  
>> timeouts, IP changes
>>    etc., we would end up with an incomplete Horde session.
>>
>> horde/login.php |   10 ++++------
>> 1 files changed, 4 insertions(+), 6 deletions(-)
>>
>> http://git.horde.org/diff.php/horde/login.php?rt=horde-git&r1=35e389de60a386533e24b01adfcffe9c896eb611&r2=f393c842d15ddf88d0b89ceaffc2b27a1e19595b
>
> Authentication is needed on the login screen for things like IMP  
> authentication when hordeauth == false and Horde authentication !=  
> IMP application authentication.

In that case IMP authentication should check for Horde authentication.  
But with the logic before my fix, several authentication scenarios  
didn't work anymore as expected.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/