За последние 24 часа нас посетили 39720 программистов и 2346 роботов. Сейчас ищут 1197 программистов ...

Авторизация на токенах и регистрация

Тема в разделе "PHP для новичков", создана пользователем Dimon2x, 13 июн 2017.

  1. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Всё работает.
    Правильно ли я, использовал способ токен?

    Правильно ли использовал функцию
    PHP:
    1. ini_set('session.use_only_cookies', true);
    и
    PHP:
    1. output_add_rewrite_var('token', $token);
    А вот это нормальный способ,на 136 строке?
    PHP:
    1. $tokenstr = strval(date('s')) . $salt;
    А саму регистрацию нормально сделал?

    А что мне дают эти токены?

    А как сделать, что бы после закрытия браузера, сессия сохранялась?

    PHP:
    1. <?php
    2. ini_set('session.use_only_cookies', true);
    5.  
    6. $host = 'localhost';
    7. $db = 'token';
    8. $user = 'root';
    9. $pass = '';
    10. $charset = 'utf8';
    11.  
    12. $dsn = "mysql:host=$host; dbname=$db;charset=$charset";
    13.  
    14. $opt = [
    15.     PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
    16.     PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
    17.     PDO::ATTR_EMULATE_PREPARES   => false,
    18.     ];
    19.    
    20. $pdo = new PDO($dsn, $user, $pass, $opt);
    21.  
    22.  
    23. ?>
    24. <!DOCTYPE html>
    25. <html>
    26.     <head>
    27.         <title>Токен</title>
    28.         <meta charset="utf-8">
    29.         <style>
    30.             .mudak {
    31.                 color: red;
    32.                 font-weight: bold;
    33.             }
    34.         </style>
    35.     </head>
    36.     <body>
    37.     <?php
    38.         if(!empty($_GET['action'])) {
    39.             if($_GET['action'] == 'exit') {
    40.                 unset($_SESSION['token']);
    41.                 session_destroy();
    42.                 header('Location: ./index.php');
    43.             }
    44.         }
    45.        
    46.         if(!empty($_SESSION['token'])) {
    47.             echo '<p>Теперь у вас есть полный доступ к сайту <a href="?action=exit">Выход</a></p>';
    48.             echo $_SESSION['token'];
    49.             die();
    50.         }
    51.        
    52.         else {
    53.             echo '<p>Авторизируйтесь, что бы увидеть содержимое сайта</p>';
    54.            
    55.             require_once 'auth.php';
    56.            
    57.             echo '<p><strong>Или зарегистрируйтесь</strong></p>';
    58.            
    59.             require_once 'registr.php';
    60.            
    61.             /*
    62.             *
    63.             *
    64.             *******Начало регистрации*********
    65.             *
    66.             *
    67.             */
    68.            
    69.             if(!empty($_POST['reg'])) {
    70.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    71.                
    72.                     $login = $_POST['login'];
    73.                     $secret = 'hhJNE63';
    74.                     $password = md5($_POST['password'].$secret);
    75.                    
    76.                     $userExists = "SELECT login FROM users WHERE login = '".$login."'";
    77.                     $queryUser = $pdo->query($userExists);
    78.                     $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    79.            
    80.                 if((count($queryUser->fetchAll()) > 0)) {
    81.                     echo '<p>Такой пользователь уже существует<p>';
    82.                     die();
    83.                 }
    84.                 else {
    85.                     $newUser = "INSERT INTO users(login, password) VALUES(:login, :password)";
    86.                     $newUserPrepare = $pdo->prepare($newUser);
    87.                     $newUserPrepare->bindValue(':login', trim($login), PDO::PARAM_STR);
    88.                     $newUserPrepare->bindValue(':password', trim($password), PDO::PARAM_STR);
    89.                     $newUserPrepare->execute();
    90.                     echo '<p>Регистрация завершена</p>';
    91.                 }
    92.                 }
    93.                
    94.                 else {
    95.                     echo '<p class="mudak">Заполните все поля</p>';
    96.                 }
    97.             }
    98.            
    99.             /*
    100.             *
    101.             *
    102.             *******Конец регистрации*********
    103.             *
    104.             *
    105.             */
    106.            
    107.            
    108.             /*
    109.             *
    110.             *
    111.             *******Начало авторизации*********
    112.             *
    113.             *
    114.             */
    115.            
    116.             if(!empty($_POST['auth'])) {
    117.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    118.                
    119.                     $login = $_POST['login'];
    120.                     $secret = 'hhJNE63';
    121.                     $password = md5($_POST['password'].$secret);
    122.                
    123.                     $user = "SELECT `login`, `password` FROM users WHERE login = :login AND password = :password";
    124.                     $resUser = $pdo->prepare($user);
    125.                     $resUser->bindValue(':login', trim($login), PDO::PARAM_STR);
    126.                     $resUser->bindValue(':password', trim($password), PDO::PARAM_STR);
    127.                    
    128.                     $resUser->execute();
    129.                     $resUser2 = $resUser->fetchAll();
    130.                    
    131.                     if(count($resUser2) === 0){
    132.                         die('Неверный логин или пароль');
    133.                     }
    134.                     else {
    135.                         $salt = $login;
    136.                         $tokenstr = strval(date('s')) . $salt;
    137.                         $token = md5($tokenstr);
    138.                        
    139.                         $_SESSION['token'] = $token;
    140.                         $_SESSION['user'] = $login;
    141.                         output_add_rewrite_var('token', $token);
    142.                         header('Location: ./index.php');
    143.                    
    144.                     }
    145.                 }
    146.                
    147.                 else {
    148.                     echo '<p class="mudak">Заполните все поля</p>';
    149.                 }
    150.             }
    151.            
    152.             /*
    153.             *
    154.             *
    155.             *******Конец авторизации*********
    156.             *
    157.             *
    158.             */
    159.         }
    160.     ?>
    161.     </body>
    162. </html>  
     
    #1 Dimon2x, 13 июн 2017
  2. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Так, как ты написал, токены бесполезны. От них есть толк, если потом их везде за собой таскать по запросам, и каждый раз проверять соответствие токена, пришедшего в запросе, тому, что в сессии. Защищает от CSRF-атак.

    Записать в базу и в куку случайную строку, сравнивать, если пришла правильная кука - авторизировать пользователя.
     
    #2 mkramer, 13 июн 2017
  3. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    А зачем в базу записывать?
    --- Добавлено ---
    не понял, а потом эту строку каждый раз записывать в БД?
     
    #3 Dimon2x, 13 июн 2017
  4. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ну чтоб знать, какому пользователю принадлежит этот токен
     
    #4 mkramer, 13 июн 2017
  5. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Я вот так сделал, только теперь, при каждом обновлении страницы, меняется название сессии

    PHP:
    1. <?php
    2. ini_set('session.use_only_cookies', true);
    5.  
    6. $host = 'localhost';
    7. $db = 'token';
    8. $user = 'root';
    9. $pass = '';
    10. $charset = 'utf8';
    11.  
    12. $dsn = "mysql:host=$host; dbname=$db;charset=$charset";
    13.  
    14. $opt = [
    15.     PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
    16.     PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
    17.     PDO::ATTR_EMULATE_PREPARES   => false,
    18.     ];
    19.    
    20. $pdo = new PDO($dsn, $user, $pass, $opt);
    21.  
    22.  
    23. ?>
    24. <!DOCTYPE html>
    25. <html>
    26.     <head>
    27.         <title>Токен</title>
    28.         <meta charset="utf-8">
    29.         <style>
    30.             .mudak {
    31.                 color: red;
    32.                 font-weight: bold;
    33.             }
    34.         </style>
    35.     </head>
    36.     <body>
    37.     <?php
    38.         if(!empty($_GET['action'])) {
    39.             if($_GET['action'] == 'exit') {
    40.                 unset($_SESSION['token']);
    41.                 session_destroy();
    42.                 setcookie ("user", "", time()-3600);
    43.                 header('Location: ./index.php');
    44.             }
    45.         }
    46.        
    47.         if(!empty($_SESSION['token']) || !empty($_COOKIE['user'])) {
    48.             $sql = "SELECT login, password FROM users WHERE randomSTR = '" .  $_COOKIE['user'] . "'";
    49.            
    50.             $queryUser = $pdo->query($sql);
    51.             $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    52.             $userArray = $queryUser->fetchAll();
    53.            
    54.             if(!empty($userArray[0]['login'])) {
    55.                 $login = $userArray[0]['login'];
    56.                
    57.                 $salt = $login;
    58.                 $tokenstr = strval(date('s')) . $salt;
    59.                 $token = md5($tokenstr);
    60.                        
    61.                 $_SESSION['token'] = $token;
    62.                 output_add_rewrite_var('token', $token);
    63.             }
    64.            
    65.             else {
    66.                 die('<p>Авторизируйтесь, что бы увидеть содержимое сайта</p>');
    67.             }
    68.            
    69.            
    70.             echo '<p>Теперь у вас есть полный доступ к сайту <a href="?action=exit">Выход</a></p>';
    71.            
    72.             echo $_SESSION['token'];
    73.            
    74.         }
    75.        
    76.         else {
    77.             echo '<p>Авторизируйтесь, что бы увидеть содержимое сайта</p>';
    78.            
    79.             require_once 'auth.php';
    80.            
    81.             echo '<p><strong>Или зарегистрируйтесь</strong></p>';
    82.            
    83.             require_once 'registr.php';
    84.            
    85.             /*
    86.             *
    87.             *
    88.             *******Начало регистрации*********
    89.             *
    90.             *
    91.             */
    92.            
    93.             if(!empty($_POST['reg'])) {
    94.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    95.                
    96.                     $login = $_POST['login'];
    97.                     $secret = 'hhJNE63';
    98.                     $password = md5($_POST['password'].$secret);
    99.                    
    100.                     $userExists = "SELECT login FROM users WHERE login = '".$login."'";
    101.                     $queryUser = $pdo->query($userExists);
    102.                     $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    103.            
    104.                 if((count($queryUser->fetchAll()) > 0)) {
    105.                     echo '<p>Такой пользователь уже существует<p>';
    106.                     die();
    107.                 }
    108.                 else {
    109.                     $newUser = "INSERT INTO users(login, password) VALUES(:login, :password)";
    110.                     $newUserPrepare = $pdo->prepare($newUser);
    111.                     $newUserPrepare->bindValue(':login', trim($login), PDO::PARAM_STR);
    112.                     $newUserPrepare->bindValue(':password', trim($password), PDO::PARAM_STR);
    113.                     $newUserPrepare->execute();
    114.                     echo '<p>Регистрация завершена</p>';
    115.                 }
    116.                 }
    117.                
    118.                 else {
    119.                     echo '<p class="mudak">Заполните все поля</p>';
    120.                 }
    121.             }
    122.            
    123.             /*
    124.             *
    125.             *
    126.             *******Конец регистрации*********
    127.             *
    128.             *
    129.             */
    130.            
    131.            
    132.             /*
    133.             *
    134.             *
    135.             *******Начало авторизации*********
    136.             *
    137.             *
    138.             */
    139.            
    140.             if(!empty($_POST['auth'])) {
    141.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    142.                
    143.                     $login = $_POST['login'];
    144.                     $secret = 'hhJNE63';
    145.                     $password = md5($_POST['password'].$secret);
    146.                
    147.                     $user = "SELECT `login`, `password` FROM users WHERE login = :login AND password = :password";
    148.                     $resUser = $pdo->prepare($user);
    149.                     $resUser->bindValue(':login', trim($login), PDO::PARAM_STR);
    150.                     $resUser->bindValue(':password', trim($password), PDO::PARAM_STR);
    151.                    
    152.                     $resUser->execute();
    153.                     $resUser2 = $resUser->fetchAll();
    154.                    
    155.                     if(count($resUser2) === 0){
    156.                         die('Неверный логин или пароль');
    157.                     }
    158.                     else {
    159.                         $arr = ['f', 3, 'g', 'bd', 5, 'p', 'k', 'n', 'v', 'w', 33, 'c', 'o'];
    160.                         shuffle($arr);
    161.                        
    162.                         $str = '';
    163.                        
    164.                         foreach($arr as $k) {
    165.                             $str .= $k;
    166.                         }
    167.                        
    168.                         $sql = "UPDATE users SET randomSTR ='" . $str . "' WHERE login = '" . $login . "'";
    169.                        
    170.                         $queryToket = $pdo->query($sql);
    171.                        
    172.                         setcookie ("user", $str, time()+3600);
    173.                        
    174.                        
    175.                         $salt = $login;
    176.                         $tokenstr = strval(date('s')) . $salt;
    177.                         $token = md5($tokenstr);
    178.                        
    179.                         $_SESSION['token'] = $token;
    180.                         output_add_rewrite_var('token', $token);
    181.                         header('Location: ./index.php');
    182.                    
    183.                     }
    184.                 }
    185.                
    186.                 else {
    187.                     echo '<p class="mudak">Заполните все поля</p>';
    188.                 }
    189.             }
    190.            
    191.             /*
    192.             *
    193.             *
    194.             *******Конец авторизации*********
    195.             *
    196.             *
    197.             */
    198.         }
    199.     ?>
    200.     </body>
    201. </html>  
    --- Добавлено ---
    Изменил на это

    PHP:
    1. if(empty($_SESSION['token']) && !empty($_COOKIE['user'])) {
    2.             $sql = "SELECT login, password FROM users WHERE randomSTR = '" .  $_COOKIE['user'] . "'";
    3.            
    4.             $queryUser = $pdo->query($sql);
    5.             $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    6.             $userArray = $queryUser->fetchAll();
    7.            
    8.             if(!empty($userArray[0]['login'])) {
    9.                 $login = $userArray[0]['login'];
    10.                
    11.                 $salt = $login;
    12.                 $tokenstr = strval(date('s')) . $salt;
    13.                 $token = md5($tokenstr);
    14.                        
    15.                 $_SESSION['token'] = $token;
    16.                 output_add_rewrite_var('token', $token);
    17.             }
    18.         }
    19.        
    20.         if(!empty($_SESSION['token']) ) {
    21.            
    22.             echo '<p>Теперь у вас есть полный доступ к сайту <a href="?action=exit">Выход</a></p>';
    23.            
    24.             echo $_SESSION['token'];
    25.            
    26.         }
    --- Добавлено ---
    Но теперь получается, хакер может подставлять значение в куку и если оно совпадёт, то произойдёт авторизация
     
    #5 Dimon2x, 13 июн 2017
  6. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ему ещё надо его откуда-то взять, поэтому мы генерируем случайную строку. Но код у тебя странный
     
    #6 mkramer, 13 июн 2017
  7. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Почему странный?
     
    #7 Dimon2x, 13 июн 2017
  8. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Вот так сделал удаление статьи, с помощью токена, правильно ли

    PHP:
    1. <?php
    2.  
    3. echo '<p>Добавить статью</p>';
    4.  
    5. require_once 'add.php';
    6.  
    7. echo '<p>Мои статьи</p>';
    8.  
    9. $myArticle = "SELECT article.id, title, user_id FROM article INNER JOIN users ON article.user_id = " . $_SESSION['id'] . " GROUP BY article.title";
    10. $sth = $pdo->query($myArticle);
    11. $sth->setFetchMode(PDO::FETCH_ASSOC);
    12. $userArticle = $sth->fetchAll();
    13.  
    14. echo '<pre>';
    15. print_r($userArticle);
    16. echo '</pre>';
    17.  
    18.  
    19. echo '<table>';
    20. echo '<tr>';
    21.     echo '<td>Название</td>';
    22.     echo '<td>Действие</td>';
    23. echo '</tr>';
    24.  
    25. foreach($userArticle as $key) {
    26.     echo '<tr>';
    27.     echo '<td>'.$key['title'].'</td>';
    28.     echo '<td><form method="post">
    29.        <input type="hidden" name="id-title" value="'.$key['id'].'">
    30.        <input type="hidden" name="id-user" value="'.$key['user_id'].'">
    31.        <input type="submit" name="del-article" value="Удалить">
    32.    </form></td>';
    33.     echo '</tr>';
    34. }
    35. echo '<table>';
    36.  
    37. if(!empty($_POST['del-article'])) {
    38.    
    39.     $sql = "SELECT article.id, `title`, `user_id`, users.randomSTR FROM article INNER JOIN users ON article.id = '".$_POST['id-title']."' AND article.user_id = '".$_POST['id-user']."' AND users.randomSTR = '".$_COOKIE['user']."' GROUP BY article.title";
    40.     $sth = $pdo->query($sql);
    41.     $sth->setFetchMode(PDO::FETCH_ASSOC);
    42.     $delArticle = $sth->fetchAll();
    43.    
    44.     if(count($delArticle[0]) > 1) {
    45.         $sqlDelArticle = "DELETE FROM article WHERE id = :id";
    46.         $del = $pdo->prepare($sqlDelArticle);
    47.         $del->bindValue(':id', $_POST['id-title'], PDO::PARAM_INT);
    48.         $del->execute();
    49.         echo 'Удалено';
    50.     }
    51.    
    52.     else {
    53.         echo 'Ошибка удаления';
    54.     }
    55.    
    56.     /*
    57.     echo '<pre>';
    58.     print_r($delArticle);
    59.     echo '<pre>';
    60.     echo count($delArticle[0]);
    61.     */
    62.    
    63.    
    64. }
    65.  
    66.  
    67. ///SELECT article.id, title, user_id FROM article LEFT JOIN users ON article.user_id = 3 GROUP BY article.title;
     
    #8 Dimon2x, 14 июн 2017
  9. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Нафига ты токен создавал, если нигде его не проверяешь?
    --- Добавлено ---
    Весь смысл в том, что
    PHP:
    1. if ($_GET["token"] !== $_SESSION["token"]) {
    2.     unset($_SESSION["user_id"]);
    3.     header("Location: /login.php");
    4.     die;
    5. }
    Без этого пользы от его генерации нету.
    --- Добавлено ---
    Это вообще бред. У тебя удалять могут только те, кто поставил галочку "запомнить меня"? Или у тебя все запоминаются? Ты путаешь назначения инструментов - случайной строки в куках (для того, чтобы пользователь автоматом перелогинивался при каждом заходе на сайт) и токена сессии (для того, чтобы никто не подменил запрос)
     
    #9 mkramer, 14 июн 2017
  10. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Пока что, все запоминаются.

    Как пользоваться токенами, я так и не понял
    --- Добавлено ---
    А сейчас же, уже не могут подменить запрос
     
    #10 Dimon2x, 14 июн 2017
  11. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Изменил запись токена в регистрации и в авторизации

    Так правильно?

    Удаление сделал по условию

    PHP:
    1. if(!empty($_POST['del-article'])) {
    2.    
    3.         if($_POST['token'] == $_SESSION['token']) {
    PHP:
    1. /*
    2.             *
    3.             *
    4.             *******Начало регистрации*********
    5.             *
    6.             *
    7.             */
    8.            
    9.             if(!empty($_POST['reg'])) {
    10.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    11.                
    12.                     $login = $_POST['login'];
    13.                     $secret = 'hhJNE63';
    14.                     $password = md5($_POST['password'].$secret);
    15.                    
    16.                     $userExists = "SELECT login FROM users WHERE login = '".$login."'";
    17.                     $queryUser = $pdo->query($userExists);
    18.                     $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    19.            
    20.                 if((count($queryUser->fetchAll()) > 0)) {
    21.                     echo '<p>Такой пользователь уже существует<p>';
    22.                     die();
    23.                 }
    24.                 else {
    25.                     $newUser = "INSERT INTO users(login, password) VALUES(:login, :password)";
    26.                     $newUserPrepare = $pdo->prepare($newUser);
    27.                     $newUserPrepare->bindValue(':login', trim($login), PDO::PARAM_STR);
    28.                     $newUserPrepare->bindValue(':password', trim($password), PDO::PARAM_STR);
    29.                     $newUserPrepare->execute();
    30.                     echo '<p>Регистрация завершена</p>';
    31.                 }
    32.                 }
    33.                
    34.                 else {
    35.                     echo '<p class="mudak">Заполните все поля</p>';
    36.                 }
    37.             }
    38.            
    39.             /*
    40.             *
    41.             *
    42.             *******Конец регистрации*********
    43.             *
    44.             *
    45.             */
    46.            
    47.            
    48.             /*
    49.             *
    50.             *
    51.             *******Начало авторизации*********
    52.             *
    53.             *
    54.             */
    55.            
    56.             if(!empty($_POST['auth'])) {
    57.                 if(!empty($_POST['login']) && !empty($_POST['password'])) {
    58.                
    59.                     $login = $_POST['login'];
    60.                     $secret = 'hhJNE63';
    61.                     $password = md5($_POST['password'].$secret);
    62.                
    63.                     $user = "SELECT `id`, `login`, `password` FROM users WHERE login = :login AND password = :password";
    64.                     $resUser = $pdo->prepare($user);
    65.                     $resUser->bindValue(':login', trim($login), PDO::PARAM_STR);
    66.                     $resUser->bindValue(':password', trim($password), PDO::PARAM_STR);
    67.                    
    68.                     $resUser->execute();
    69.                     $resUser2 = $resUser->fetchAll();
    70.                     $userId = $resUser2[0]['id'];
    71.                    
    72.                     if(count($resUser2) === 0){
    73.                         die('Неверный логин или пароль');
    74.                     }
    75.                     else {
    76.                        
    77.                         $salt = $login;
    78.                         $tokenstr = strval(date('s')) . $salt;
    79.                         $token = md5($tokenstr);
    80.                        
    81.                        
    82.                         $sql = "UPDATE users SET token ='" . $token . "' WHERE login = '" . $login . "'";
    83.                        
    84.                         $queryToket = $pdo->query($sql);
    85.                        
    86.                         setcookie ("user", $token, time()+3600);
    87.                        
    88.                        
    89.                         $_SESSION['token'] = $token;
    90.                         $_SESSION['user'] = $login;
    91.                         $_SESSION['id'] = $userId;
    92.                         output_add_rewrite_var('token', $token);
    93.                         header('Location: ./index.php');
    94.                    
    95.                     }
    96.                 }
    PHP:
    1. <?php
    2.  
    3. echo '<p>Добавить статью</p>';
    4.  
    5. require_once 'add.php';
    6.  
    7. echo '<p>Мои статьи</p>';
    8.  
    9. $myArticle = "SELECT article.id, title, user_id FROM article INNER JOIN users ON article.user_id = " . $_SESSION['id'] . " GROUP BY article.title";
    10. $sth = $pdo->query($myArticle);
    11. $sth->setFetchMode(PDO::FETCH_ASSOC);
    12. $userArticle = $sth->fetchAll();
    13.  
    14. echo '<table>';
    15. echo '<tr>';
    16.     echo '<td>Название</td>';
    17.     echo '<td>Действие</td>';
    18. echo '</tr>';
    19.  
    20. foreach($userArticle as $key) {
    21.     echo '<tr>';
    22.     echo '<td>'.$key['title'].'</td>';
    23.     echo '<td><form method="post">
    24.        <input type="hidden" name="id-title" value="'.$key['id'].'">
    25.        <input type="hidden" name="token" value="' .$_SESSION['token'].'">
    26.        <input type="submit" name="del-article" value="Удалить">
    27.    </form></td>';
    28.     echo '</tr>';
    29. }
    30. echo '<table>';
    31.  
    32.  
    33.     if(!empty($_POST['del-article'])) {
    34.    
    35.         if($_POST['token'] == $_SESSION['token']) {
    36.    
    37.         $sql = "SELECT article.id, `title`, `user_id`, users.id, users.token FROM article INNER JOIN users ON article.id = '".$_POST['id-title']."' AND article.user_id = '".$_SESSION['id']."' AND users.token = '".$_COOKIE['user']."' AND users.id = '".$_SESSION['id']."' GROUP BY article.title";
    38.         $sth = $pdo->query($sql);
    39.         $sth->setFetchMode(PDO::FETCH_ASSOC);
    40.         $delArticle = $sth->fetchAll();
    41.    
    42.         if(count($delArticle[0]) > 1) {
    43.             $sqlDelArticle = "DELETE FROM article WHERE id = :id";
    44.             $del = $pdo->prepare($sqlDelArticle);
    45.             $del->bindValue(':id', $_POST['id-title'], PDO::PARAM_INT);
    46.             $del->execute();
    47.             echo 'Удалено';
    48.         }
    49.        
    50.         else {
    51.             echo 'Ошибка удаления';
    52.         }
    53.     }
    54.    
    55. }
     
    #11 Dimon2x, 14 июн 2017
  12. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Вот это уже больше похоже на правду. ТОлько строгое равенство сделай
    Но вообще, если ты уж завёл token в сессии, таскай его вообще во всех ссылках и проверяй в каждом запросе.

    Строго говоря, если я у тебя авторизировался, то во все ссылки на сайте добавляется example.com/url?token=jfsjfghlskjghsdlljhjhlafd, и каждый раз, помимо авторизации пользователя, ты проверяешь, передан ли этот токен в гете. И если не передан, разавторизируешь сразу. Можно ещё для них всякие сроки жизни придумывать и т.п.
     
    #12 mkramer, 14 июн 2017
    Dimon2x нравится это.
  13. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Сделал регистрацию, что бы нельзя было допустить sql инъекцию, нормально ли?

    PHP:
    1.     function registr($login, $password, $email) {
    2.         $input_email = filter_var($email, FILTER_VALIDATE_EMAIL);
    3.         $pass = password_hash($passwod, PASSWORD_DEFAULT);
    4.  
    5.        
    6.         if($input_email === false) {
    7.             echo '<p>Введите правильно email</p>';
    8.             die();
    9.         }
    10.        
    11.         $newLogin = str_replace(";","",$login);
    12.         $newLogin2 = str_replace(")","",$newLogin);
    13.         $newLogin3 = str_replace("'","",$newLogin2);
    14.        
    15.         $userExists = "SELECT DISTINCT login FROM users WHERE login = '".$newLogin3."'";
    16.         $queryUser = $this->db->query($userExists);
    17.         $queryUser->setFetchMode(PDO::FETCH_ASSOC);
    18.        
    19.         if((count($queryUser->fetchAll()) > 0)) {
    20.             echo '<p>Такой пользователь уже существует</p>';
    21.             die();
    22.         }
    23.        
    24.  
    25.         else {
    26.             $newUser = "INSERT INTO users(login, password, email) VALUES(:login, :password, :email)";
    27.             $newUserPrepare = $this->db->prepare($newUser);
    28.             $newUserPrepare->bindValue(':login', trim($newLogin3), PDO::PARAM_STR);
    29.             $newUserPrepare->bindValue(':password', trim($pass), PDO::PARAM_STR);
    30.             $newUserPrepare->bindValue(':email', trim($input_email), PDO::PARAM_STR);
    31.             $newUserPrepare->execute();
    32.             echo '<p>Регистрация завершена</p>';
    33.             echo '<p><a href="?index">Перейти на главную страницу сайта</a></p>';
    34.            
    35.         }
    36.        
    37.     }
     
    #13 Dimon2x, 14 июн 2017
  14. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Вроде умеешь подготовленными запросами пользоваться, нахрена такую чушь рисовать?
     
    #14 mkramer, 14 июн 2017
  15. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    подготовленные запросы, эти символы не удаляют
     
    #15 Dimon2x, 14 июн 2017
  16. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    А зачем их удалять?
     
    #16 mkramer, 14 июн 2017
  17. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Потому что могут зарегистрировать такой ник Evil'); DROP TABLE drop_table;--
     
    #17 Dimon2x, 14 июн 2017
  18. Алекс8

    Алекс8 Активный пользователь

    С нами с:
    18 май 2017
    Сообщения:
    1.730
    Симпатии:
    359
    если запрос слать отдельно а данные отдельно то у него такой ник запишется в базу))
     
    #18 Алекс8, 14 июн 2017
  19. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Ну и пусть. Это не сработает, если запросы подготовлены
     
    #19 mkramer, 14 июн 2017
  20. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Сделал авторизацию, с помощью password_verify и после авторизации, сразу добавил токен в GET параметр, так правильно?

    PHP:
    1.     function Login($login, $password) {
    2.         $newLogin = str_replace(";","",$login);
    3.         $newLogin2 = str_replace(")","",$newLogin);
    4.         $newLogin3 = str_replace("'","",$newLogin2);
    5.        
    6.         $sqlPass = "SELECT DISTINCT `id`, `login`, `password` FROM users WHERE login = :log";
    7.         $resPass = $this->db->prepare($sqlPass);
    8.         $resPass->bindValue(':log', trim($newLogin3), PDO::PARAM_STR);
    9.         $resPass->execute();
    10.        
    11.         $allRes = $resPass->fetchAll();
    12.         $needPassword = $allRes[0]['password'];
    13.         $userId = $allRes[0]['id'];
    14.         $userLogin = $allRes[0]['login'];
    15.        
    16.        
    17.         $hash = $needPassword;
    18.  
    19.         if (password_verify($password, $hash)) {
    20.            
    21.             $salt = 'slovo';
    22.             $tokenstr = strval(date('s')) . $salt;
    23.             $token = md5($tokenstr);
    24.                        
    25.                        
    26.             $sql = "UPDATE users SET token ='" . $token . "' WHERE login = :log2";
    27.             $queryToket = $this->db->prepare($sql);
    28.             $queryToket->bindValue(':log2', trim($newLogin3), PDO::PARAM_STR);
    29.             $queryToket->execute();
    30.                        
    31.             setcookie ("user", $token, time()+604800);
    32.                    
    33.             $_SESSION['token'] = $token;
    34.             $_SESSION['userPdd'] = $userLogin;
    35.             $_SESSION['id'] = $userId;
    36.             output_add_rewrite_var('token', $token);
    37.             header('Location: ./index.php?token='.$_SESSION['token']);
    38.            
    39.            
    40.         }
    41.         else {
    42.             die('Неверный логин или пароль');
    43.         }
    44.        
    45.     }
     
    #20 Dimon2x, 14 июн 2017
  21. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    Да.
     
    #21 mkramer, 14 июн 2017
  22. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    А выход, тоже делать по токену или можно просто так?
     
    #22 Dimon2x, 14 июн 2017
  23. romach

    romach Старожил

    С нами с:
    26 окт 2013
    Сообщения:
    2.904
    Симпатии:
    719
    чуваки, отсыпьте мне тоже этих ваших токенов, а? )
     
    #23 romach, 16 июн 2017
  24. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.408
    Симпатии:
    1.768
    форум отсыпал уже
     
    #24 igordata, 16 июн 2017
(Вы должны войти или зарегистрироваться, чтобы разместить сообщение.)