From: Florent Bruneau <florent.bruneau@polytechnique.org>
Date: Sun, 10 Aug 2008 19:38:55 +0000 (+0200)
Subject: Don't log 'cookie-on' each time somebody log-in with a cookie.
X-Git-Tag: xorg/0.10.0~138
X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=d0621f88bdfeabaab38da58a95a6b94f71e011c5;p=platal.git

Don't log 'cookie-on' each time somebody log-in with a cookie.

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>
---

diff --git a/classes/xorgsession.php b/classes/xorgsession.php
index 229df76..c259d21 100644
--- a/classes/xorgsession.php
+++ b/classes/xorgsession.php
@@ -223,10 +223,10 @@ class XorgSession extends PlSession
             $logger = S::logger($uid);
             setcookie('ORGuid', $uid, (time() + 25920000), '/', '', 0);
 
-            if (S::i('auth_by_cookie') == S::i('uid') || Post::v('remember', 'false') == 'true') {
+            if (S::i('auth_by_cookie') == $uid || Post::v('remember', 'false') == 'true') {
                 $cookie = hash_encrypt($sess['password']);
                 setcookie('ORGaccess', $cookie, (time() + 25920000), '/', '', 0);
-                if ($logger) {
+                if ($logger && S::i('auth_by_cookie') != $uid) {
                     $logger->log("cookie_on");
                 }
             } else {
@@ -243,6 +243,9 @@ class XorgSession extends PlSession
         $this->setSkin();
         $this->updateNbNotifs();
         check_redirect();
+
+        // We should not have to use this private data anymore
+        S::kill('auth_by_cookie');
         return true;
     }