]> git.mxchange.org Git - friendica.git/blobdiff - include/security.php
added spaces + some curly braces + some usage of dbm::is_result()
[friendica.git] / include / security.php
index cd00b5f7b0bbc36a75f511c8f865fe643164f6b7..ac462d1df1171d21f9cb87302b451711fe90f0d1 100644 (file)
@@ -1,5 +1,46 @@
 <?php
 
+use Friendica\App;
+
+/**
+ * @brief Calculate the hash that is needed for the "Friendica" cookie
+ *
+ * @param array $user Record from "user" table
+ *
+ * @return string Hashed data
+ */
+function cookie_hash($user) {
+       return(hash("sha256", get_config("system", "site_prvkey").
+                               $user["uprvkey"].
+                               $user["password"]));
+}
+
+/**
+ * @brief Set the "Friendica" cookie
+ *
+ * @param int $time
+ * @param array $user Record from "user" table
+ */
+function new_cookie($time, $user = array()) {
+
+       if ($time != 0) {
+               $time = $time + time();
+       }
+
+       if ($user) {
+               $value = json_encode(array("uid" => $user["uid"],
+                                       "hash" => cookie_hash($user),
+                                       "ip" => $_SERVER['REMOTE_ADDR']));
+       }
+       else {
+               $value = "";
+       }
+
+       setcookie("Friendica", $value, $time, "/", "",
+               (get_config('system', 'ssl_policy') == SSL_POLICY_FULL), true);
+
+}
+
 function authenticate_success($user_record, $login_initial = false, $interactive = false, $login_refresh = false) {
 
        $a = get_app();
@@ -15,58 +56,65 @@ function authenticate_success($user_record, $login_initial = false, $interactive
 
        $a->user = $user_record;
 
-       if($interactive) {
-               if($a->user['login_date'] === '0000-00-00 00:00:00') {
+       if ($interactive) {
+               if ($a->user['login_date'] <= NULL_DATE) {
                        $_SESSION['return_url'] = 'profile_photo/new';
                        $a->module = 'profile_photo';
                        info( t("Welcome ") . $a->user['username'] . EOL);
                        info( t('Please upload a profile photo.') . EOL);
-               }
-               else
+               } else {
                        info( t("Welcome back ") . $a->user['username'] . EOL);
+               }
        }
 
        $member_since = strtotime($a->user['register_date']);
-       if(time() < ($member_since + ( 60 * 60 * 24 * 14)))
+       if (time() < ($member_since + ( 60 * 60 * 24 * 14))) {
                $_SESSION['new_member'] = true;
-       else
+       } else {
                $_SESSION['new_member'] = false;
-       if(strlen($a->user['timezone'])) {
+       }
+       if (strlen($a->user['timezone'])) {
                date_default_timezone_set($a->user['timezone']);
                $a->timezone = $a->user['timezone'];
        }
 
        $master_record = $a->user;
 
-       if((x($_SESSION,'submanage')) && intval($_SESSION['submanage'])) {
-               $r = q("select * from user where uid = %d limit 1",
+       if ((x($_SESSION,'submanage')) && intval($_SESSION['submanage'])) {
+               $r = q("SELECT * FROM `user` WHERE `uid` = %d LIMIT 1",
                        intval($_SESSION['submanage'])
                );
-               if (dbm::is_result($r))
+               if (dbm::is_result($r)) {
                        $master_record = $r[0];
+               }
        }
 
        $r = q("SELECT `uid`,`username`,`nickname` FROM `user` WHERE `password` = '%s' AND `email` = '%s' AND `account_removed` = 0 ",
                dbesc($master_record['password']),
                dbesc($master_record['email'])
        );
-       if (dbm::is_result($r))
+       if (dbm::is_result($r)) {
                $a->identities = $r;
-       else
+       } else {
                $a->identities = array();
+       }
 
-       $r = q("select `user`.`uid`, `user`.`username`, `user`.`nickname`
-               from manage INNER JOIN user on manage.mid = user.uid where `user`.`account_removed` = 0
-               and `manage`.`uid` = %d",
+       $r = q("SELECT `user`.`uid`, `user`.`username`, `user`.`nickname`
+               FROM `manage`
+               INNER JOIN `user` ON `manage`.`mid` = `user`.`uid`
+               WHERE `user`.`account_removed` = 0 AND `manage`.`uid` = %d",
                intval($master_record['uid'])
        );
-       if (dbm::is_result($r))
+       if (dbm::is_result($r)) {
                $a->identities = array_merge($a->identities,$r);
+       }
 
-       if($login_initial)
+       if ($login_initial) {
                logger('auth_identities: ' . print_r($a->identities,true), LOGGER_DEBUG);
-       if($login_refresh)
+       }
+       if ($login_refresh) {
                logger('auth_identities refresh: ' . print_r($a->identities,true), LOGGER_DEBUG);
+       }
 
        $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `self` = 1 LIMIT 1",
                intval($_SESSION['uid']));
@@ -78,7 +126,7 @@ function authenticate_success($user_record, $login_initial = false, $interactive
 
        header('X-Account-Management-Status: active; name="' . $a->user['username'] . '"; id="' . $a->user['nickname'] .'"');
 
-       if($login_initial || $login_refresh) {
+       if ($login_initial || $login_refresh) {
 
                q("UPDATE `user` SET `login_date` = '%s' WHERE `uid` = %d",
                        dbesc(datetime_convert()),
@@ -94,6 +142,21 @@ function authenticate_success($user_record, $login_initial = false, $interactive
 
 
        }
+
+       if ($login_initial) {
+               // 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 ($_SESSION['remember']) {
+                       logger('Injecting cookie for remembered user '. $_SESSION['remember_user']['nickname']);
+                       new_cookie(604800, $user_record);
+                       unset($_SESSION['remember']);
+               }
+       }
+
+
+
        if ($login_initial) {
                call_hooks('logged_in', $a->user);
 
@@ -106,7 +169,7 @@ function authenticate_success($user_record, $login_initial = false, $interactive
 
 
 
-function can_write_wall(&$a,$owner) {
+function can_write_wall(App $a, $owner) {
 
        static $verified = 0;
 
@@ -148,8 +211,8 @@ function can_write_wall(&$a,$owner) {
                                return false;
                        }
 
-                       $r = q("SELECT `contact`.*, `user`.`page-flags` FROM `contact` INNER JOIN `user` on `user`.`uid` = `contact`.`uid` 
-                               WHERE `contact`.`uid` = %d AND `contact`.`id` = %d AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0 
+                       $r = q("SELECT `contact`.*, `user`.`page-flags` FROM `contact` INNER JOIN `user` on `user`.`uid` = `contact`.`uid`
+                               WHERE `contact`.`uid` = %d AND `contact`.`id` = %d AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
                                AND `user`.`blockwall` = 0 AND `readonly` = 0  AND ( `contact`.`rel` IN ( %d , %d ) OR `user`.`page-flags` = %d ) LIMIT 1",
                                intval($owner),
                                intval($cid),
@@ -183,31 +246,31 @@ function permissions_sql($owner_id,$remote_verified = false,$groups = null) {
         * default permissions - anonymous user
         */
 
-       $sql = " AND allow_cid = '' 
-                        AND allow_gid = '' 
-                        AND deny_cid  = '' 
-                        AND deny_gid  = '' 
+       $sql = " AND allow_cid = ''
+                        AND allow_gid = ''
+                        AND deny_cid  = ''
+                        AND deny_gid  = ''
        ";
 
        /**
         * Profile owner - everything is visible
         */
 
-       if(($local_user) && ($local_user == $owner_id)) {
-               $sql = ''; 
+       if (($local_user) && ($local_user == $owner_id)) {
+               $sql = '';
        }
 
        /**
-        * Authenticated visitor. Unless pre-verified, 
+        * Authenticated visitor. Unless pre-verified,
         * check that the contact belongs to this $owner_id
         * and load the groups the visitor belongs to.
         * If pre-verified, the caller is expected to have already
         * done this and passed the groups into this function.
         */
 
-       elseif($remote_user) {
+       elseif ($remote_user) {
 
-               if(! $remote_verified) {
+               if (! $remote_verified) {
                        $r = q("SELECT id FROM contact WHERE id = %d AND uid = %d AND blocked = 0 LIMIT 1",
                                intval($remote_user),
                                intval($owner_id)
@@ -217,18 +280,18 @@ function permissions_sql($owner_id,$remote_verified = false,$groups = null) {
                                $groups = init_groups_visitor($remote_user);
                        }
                }
-               if($remote_verified) {
+               if ($remote_verified) {
 
                        $gs = '<<>>'; // should be impossible to match
 
-                       if(is_array($groups) && count($groups)) {
-                               foreach($groups as $g)
+                       if (is_array($groups) && count($groups)) {
+                               foreach ($groups as $g)
                                        $gs .= '|<' . intval($g) . '>';
-                       } 
+                       }
 
                        /*$sql = sprintf(
-                               " AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' ) 
-                                 AND ( deny_cid  = '' OR  NOT deny_cid REGEXP '<%d>' ) 
+                               " AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' )
+                                 AND ( deny_cid  = '' OR  NOT deny_cid REGEXP '<%d>' )
                                  AND ( allow_gid = '' OR allow_gid REGEXP '%s' )
                                  AND ( deny_gid  = '' OR NOT deny_gid REGEXP '%s')
                                ",
@@ -275,21 +338,21 @@ function item_permissions_sql($owner_id,$remote_verified = false,$groups = null)
         * Profile owner - everything is visible
         */
 
-       if($local_user && ($local_user == $owner_id)) {
+       if ($local_user && ($local_user == $owner_id)) {
                $sql = '';
        }
 
        /**
-        * Authenticated visitor. Unless pre-verified, 
+        * Authenticated visitor. Unless pre-verified,
         * check that the contact belongs to this $owner_id
         * and load the groups the visitor belongs to.
         * If pre-verified, the caller is expected to have already
         * done this and passed the groups into this function.
         */
 
-       elseif($remote_user) {
+       elseif ($remote_user) {
 
-               if(! $remote_verified) {
+               if (! $remote_verified) {
                        $r = q("SELECT id FROM contact WHERE id = %d AND uid = %d AND blocked = 0 LIMIT 1",
                                intval($remote_user),
                                intval($owner_id)
@@ -299,20 +362,20 @@ function item_permissions_sql($owner_id,$remote_verified = false,$groups = null)
                                $groups = init_groups_visitor($remote_user);
                        }
                }
-               if($remote_verified) {
+               if ($remote_verified) {
 
                        $gs = '<<>>'; // should be impossible to match
 
-                       if(is_array($groups) && count($groups)) {
-                               foreach($groups as $g)
+                       if (is_array($groups) && count($groups)) {
+                               foreach ($groups as $g)
                                        $gs .= '|<' . intval($g) . '>';
-                       } 
+                       }
 
                        $sql = sprintf(
-                               /*" AND ( private = 0 OR ( private in (1,2) AND wall = 1 AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' ) 
-                                 AND ( deny_cid  = '' OR  NOT deny_cid REGEXP '<%d>' ) 
+                               /*" AND ( private = 0 OR ( private in (1,2) AND wall = 1 AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' )
+                                 AND ( deny_cid  = '' OR  NOT deny_cid REGEXP '<%d>' )
                                  AND ( allow_gid = '' OR allow_gid REGEXP '%s' )
-                                 AND ( deny_gid  = '' OR NOT deny_gid REGEXP '%s'))) 
+                                 AND ( deny_gid  = '' OR NOT deny_gid REGEXP '%s')))
                                ",
                                intval($remote_user),
                                intval($remote_user),
@@ -345,29 +408,29 @@ function item_permissions_sql($owner_id,$remote_verified = false,$groups = null)
  *    If the new page contains by any chance external elements, then the used security token is exposed by the referrer.
  *    Actually, important actions should not be triggered by Links / GET-Requests at all, but somethimes they still are,
  *    so this mechanism brings in some damage control (the attacker would be able to forge a request to a form of this type, but not to forms of other types).
- */ 
+ */
 function get_form_security_token($typename = '') {
        $a = get_app();
-       
+
        $timestamp = time();
        $sec_hash = hash('whirlpool', $a->user['guid'] . $a->user['prvkey'] . session_id() . $timestamp . $typename);
-       
+
        return $timestamp . '.' . $sec_hash;
 }
 
 function check_form_security_token($typename = '', $formname = 'form_security_token') {
        if (!x($_REQUEST, $formname)) return false;
        $hash = $_REQUEST[$formname];
-       
+
        $max_livetime = 10800; // 3 hours
-       
+
        $a = get_app();
-       
+
        $x = explode('.', $hash);
        if (time() > (IntVal($x[0]) + $max_livetime)) return false;
-       
+
        $sec_hash = hash('whirlpool', $a->user['guid'] . $a->user['prvkey'] . session_id() . $x[0] . $typename);
-       
+
        return ($sec_hash == $x[1]);
 }
 
@@ -395,20 +458,19 @@ function check_form_security_token_ForbiddenOnErr($typename = '', $formname = 'f
 
 // Returns an array of group id's this contact is a member of.
 // This array will only contain group id's related to the uid of this
-// DFRN contact. They are *not* neccessarily unique across the entire site. 
+// DFRN contact. They are *not* neccessarily unique across the entire site.
 
 
-if(! function_exists('init_groups_visitor')) {
+if (! function_exists('init_groups_visitor')) {
 function init_groups_visitor($contact_id) {
        $groups = array();
-       $r = q("SELECT `gid` FROM `group_member` 
+       $r = q("SELECT `gid` FROM `group_member`
                WHERE `contact-id` = %d ",
                intval($contact_id)
        );
        if (dbm::is_result($r)) {
-               foreach($r as $rr)
+               foreach ($r as $rr)
                        $groups[] = $rr['gid'];
        }
        return $groups;
 }}
-