Changeset 2443 for trunk/htdocs/system/classes/user.php

Timestamp:

09/04/08 23:40:29 (4 months ago)

Author:

dmondark

Message:

Do not display errors messages for log entries just because their level is >= warning. Also display minimum information on login failures. Fixes #558 (see ticket for details) and #559.

Files:

• trunk/htdocs/system/classes/user.php (modified) (3 diffs)

trunk/htdocs/system/classes/user.php

@@ -220 +220 @@
         elseif(!is_object($user)) {
             EventLog::log( sprintf(_t('Login attempt (via authentication plugin) for non-existent user %s'), $who), 'warning', 'authentication', 'habari' );
+            Session::error('Invalid username/password');
             self::$identity = null;
             return false;
@@ -235 +236 @@
             // No such user.
             EventLog::log( sprintf(_t('Login attempt for non-existent user %s'), $who), 'warning', 'authentication', 'habari' );
+            Session::error('Invalid username/password');
             self::$identity = null;
             return false;
@@ -251 +253 @@
             // Wrong password.
             EventLog::log( sprintf(_t('Wrong password for user %s'), $user->username), 'warning', 'authentication', 'habari' );
+            Session::error('Invalid username/password');
             self::$identity = null;
             return false;