]> git.mxchange.org Git - friendica.git/blobdiff - include/auth.php
Fix ostatus bug related to only_full_group_by
[friendica.git] / include / auth.php
index 4f3d0d31bb023562ca076bd5d4e8d0a6883d791c..57e9d9bf61dc82a6f2adc21da27b92f9fb605ff8 100644 (file)
@@ -2,27 +2,27 @@
 require_once('include/security.php');
 require_once('include/datetime.php');
 
-function nuke_session() {
-
-       new_cookie(0); // make sure cookie is deleted on browser close, as a security measure
-       session_unset();
-}
-
 // When the "Friendica" cookie is set, take the value to authenticate and renew the cookie.
-if(isset($_COOKIE["Friendica"])) {
+if (isset($_COOKIE["Friendica"])) {
        $data = json_decode($_COOKIE["Friendica"]);
        if (isset($data->uid)) {
                $r = q("SELECT `user`.*, `user`.`pubkey` as `upubkey`, `user`.`prvkey` as `uprvkey`
-               FROM `user` WHERE `uid` = %d AND `blocked` = 0 AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 LIMIT 1",
+               FROM `user` WHERE `uid` = %d AND NOT `blocked` AND NOT `account_expired` AND NOT `account_removed` AND `verified` LIMIT 1",
                        intval($data->uid)
                );
 
                if ($r) {
+                       if ($data->hash != cookie_hash($r[0])) {
+                               logger("Hash for user ".$data->uid." doesn't fit.");
+                               nuke_session();
+                               goaway(z_root());
+                       }
+
                        // Renew the cookie
-                       new_cookie(604800, json_encode(array("uid" => $r[0]["uid"], "ip" => $_SERVER['REMOTE_ADDR'])));
+                       new_cookie(604800, $r[0]);
 
                        // Do the authentification if not done by now
-                       if(!isset($_SESSION) OR !isset($_SESSION['authenticated'])) {
+                       if (!isset($_SESSION) OR !isset($_SESSION['authenticated'])) {
                                authenticate_success($r[0]);
 
                                if (get_config('system','paranoia'))
@@ -35,9 +35,9 @@ if(isset($_COOKIE["Friendica"])) {
 
 // login/logout
 
-if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-params'))) || ($_POST['auth-params'] !== 'login'))) {
+if (isset($_SESSION) && x($_SESSION,'authenticated') && (!x($_POST,'auth-params') || ($_POST['auth-params'] !== 'login'))) {
 
-       if(((x($_POST,'auth-params')) && ($_POST['auth-params'] === 'logout')) || ($a->module === 'logout')) {
+       if ((x($_POST,'auth-params') && ($_POST['auth-params'] === 'logout')) || ($a->module === 'logout')) {
 
                // process logout request
                call_hooks("logging_out");
@@ -46,34 +46,34 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
                goaway(z_root());
        }
 
-       if(x($_SESSION,'visitor_id') && (! x($_SESSION,'uid'))) {
+       if (x($_SESSION,'visitor_id') && !x($_SESSION,'uid')) {
                $r = q("SELECT * FROM `contact` WHERE `id` = %d LIMIT 1",
                        intval($_SESSION['visitor_id'])
                );
-               if(count($r)) {
+               if (dbm::is_result($r)) {
                        $a->contact = $r[0];
                }
        }
 
-       if(x($_SESSION,'uid')) {
+       if (x($_SESSION,'uid')) {
 
                // already logged in user returning
 
                $check = get_config('system','paranoia');
                // extra paranoia - if the IP changed, log them out
-               if($check && ($_SESSION['addr'] != $_SERVER['REMOTE_ADDR'])) {
-                       logger('Session address changed. Paranoid setting in effect, blocking session. '
-                               $_SESSION['addr'].' != '.$_SERVER['REMOTE_ADDR']);
+               if ($check && ($_SESSION['addr'] != $_SERVER['REMOTE_ADDR'])) {
+                       logger('Session address changed. Paranoid setting in effect, blocking session. '.
+                               $_SESSION['addr'].' != '.$_SERVER['REMOTE_ADDR']);
                        nuke_session();
                        goaway(z_root());
                }
 
                $r = q("SELECT `user`.*, `user`.`pubkey` as `upubkey`, `user`.`prvkey` as `uprvkey`
-               FROM `user` WHERE `uid` = %d AND `blocked` = 0 AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 LIMIT 1",
+               FROM `user` WHERE `uid` = %d AND NOT `blocked` AND NOT `account_expired` AND NOT `account_removed` AND `verified` LIMIT 1",
                        intval($_SESSION['uid'])
                );
 
-               if(! count($r)) {
+               if (!dbm::is_result($r)) {
                        nuke_session();
                        goaway(z_root());
                }
@@ -81,10 +81,10 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
                // Make sure to refresh the last login time for the user if the user
                // stays logged in for a long time, e.g. with "Remember Me"
                $login_refresh = false;
-               if(! x($_SESSION['last_login_date'])) {
+               if (!x($_SESSION['last_login_date'])) {
                        $_SESSION['last_login_date'] = datetime_convert('UTC','UTC');
                }
-               if(strcmp(datetime_convert('UTC','UTC','now - 12 hours'), $_SESSION['last_login_date']) > 0) {
+               if (strcmp(datetime_convert('UTC','UTC','now - 12 hours'), $_SESSION['last_login_date']) > 0) {
 
                        $_SESSION['last_login_date'] = datetime_convert('UTC','UTC');
                        $login_refresh = true;
@@ -95,10 +95,10 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
 
        session_unset();
 
-       if((x($_POST,'password')) && strlen($_POST['password']))
+       if (x($_POST,'password') && strlen($_POST['password']))
                $encrypted = hash('whirlpool',trim($_POST['password']));
        else {
-               if((x($_POST,'openid_url')) && strlen($_POST['openid_url']) ||
+               if ((x($_POST,'openid_url')) && strlen($_POST['openid_url']) ||
                   (x($_POST,'username')) && strlen($_POST['username'])) {
 
                        $noid = get_config('system','no_openid');
@@ -111,7 +111,7 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
 
                        // if it's an email address or doesn't resolve to a URL, fail.
 
-                       if(($noid) || (strpos($temp_string,'@')) || (! validate_url($temp_string))) {
+                       if ($noid || strpos($temp_string,'@') || !validate_url($temp_string)) {
                                $a = get_app();
                                notice(t('Login failed.').EOL);
                                goaway(z_root());
@@ -125,17 +125,17 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
                                $openid = new LightOpenID;
                                $openid->identity = $openid_url;
                                $_SESSION['openid'] = $openid_url;
-                               $a = get_app();
-                               $openid->returnUrl = $a->get_baseurl(true).'/openid';
+                               $_SESSION['remember'] = $_POST['remember'];
+                               $openid->returnUrl = App::get_baseurl(true).'/openid';
                                goaway($openid->authUrl());
                        } catch (Exception $e) {
-                               notice(t('We encountered a problem while logging in with the OpenID you provided. Please check the correct spelling of the ID.').'<br /><br >'. t('The error message was:').' '.$e->getMessage());
+                               notice(t('We encountered a problem while logging in with the OpenID you provided. Please check the correct spelling of the ID.').'<br /><br >'.t('The error message was:').' '.$e->getMessage());
                        }
                        // NOTREACHED
                }
        }
 
-       if((x($_POST,'auth-params')) && $_POST['auth-params'] === 'login') {
+       if (x($_POST,'auth-params') && $_POST['auth-params'] === 'login') {
 
                $record = null;
 
@@ -156,20 +156,20 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
 
                call_hooks('authenticate', $addon_auth);
 
-               if(($addon_auth['authenticated']) && (count($addon_auth['user_record']))) {
+               if ($addon_auth['authenticated'] && count($addon_auth['user_record']))
                        $record = $addon_auth['user_record'];
-               else {
+               else {
 
                        // process normal login request
 
                        $r = q("SELECT `user`.*, `user`.`pubkey` as `upubkey`, `user`.`prvkey` as `uprvkey`
                                FROM `user` WHERE (`email` = '%s' OR `nickname` = '%s')
-                               AND `password` = '%s' AND `blocked` = 0 AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 LIMIT 1",
+                               AND `password` = '%s' AND NOT `blocked` AND NOT `account_expired` AND NOT `account_removed` AND `verified` LIMIT 1",
                                dbesc(trim($_POST['username'])),
                                dbesc(trim($_POST['username'])),
                                dbesc($encrypted)
                        );
-                       if(count($r))
+                       if (dbm::is_result($r))
                                $record = $r[0];
                }
 
@@ -179,29 +179,23 @@ if((isset($_SESSION)) && (x($_SESSION,'authenticated')) && ((! (x($_POST,'auth-p
                        goaway(z_root());
                }
 
-               // If the user specified to remember the authentication, then set a cookie
-               // that expires after one week (the default is when the browser is closed).
-               // The cookie will be renewed automatically.
-               // The week ensures that sessions will expire after some inactivity.
-               if($_POST['remember'])
-                       new_cookie(604800, json_encode(array("uid" => $r[0]["uid"], "ip" => $_SERVER['REMOTE_ADDR'])));
-               else
+               if (! $_POST['remember']) {
                        new_cookie(0); // 0 means delete on browser exit
+               }
 
                // if we haven't failed up this point, log them in.
-
+               $_SESSION['remember'] = $_POST['remember'];
                $_SESSION['last_login_date'] = datetime_convert('UTC','UTC');
                authenticate_success($record, true, true);
        }
 }
 
-function new_cookie($time, $value = "") {
-
-       if ($time != 0)
-               $time = $time + time();
-
-       setcookie("Friendica", $value, $time, "/", "",
-               (get_config('system', 'ssl_policy') == SSL_POLICY_FULL), true);
+/**
+ * @brief Kills the "Friendica" cookie and all session data
+ */
+function nuke_session() {
 
-       return;
+       new_cookie(-3600); // make sure cookie is deleted on browser close, as a security measure
+       session_unset();
+       session_destroy();
 }