Модераторы: skyboy, MoLeX, Aliance, ksnk

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Вопрос безопасности при авторизации 
:(
    Опции темы
NLspieler
Дата 5.2.2010, 00:02 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 619
Регистрация: 13.10.2008
Где: Берлин

Репутация: 16
Всего: 19



Цитата(yngwie19 @  4.2.2010,  21:10 Найти цитируемый пост)
Simpliest, Подскажите пожалуйста не могли бы Вы дать ссылку на статью, где будет описан правильный(с точки зрения безопасности) подход к регистрации пользователей на сайте. А то в инете много разных подходов, а какой лучше выбрать и дальше не знаю. 


Правильный подход постараюсь описать несколькими тезисами,
кто не согласен, высказать свои возмущения.

Если сайт не связан с деньгами, то никаких супер приемов для обеспечения безопасности не нужно, достаточно только 
mysql_real_escape_string

Иначе, нужно включить мозг и подумать.

1) Во-первых, нужно организовать правильную капчу - цифры, которые нужные вводить с картинки.
При правальной реализации, это хорошо защитит от брутфорса - т.е. подбора пароля методом перебора

2) При регистрации, если пользователь сам выбирает пароль, проверять не слишком ли простой пароль, и если это так, 
то просить ввести другой, более сложный.

3) Не хранить важной информации в куках, которые хранятся на машине пользователя и передаются с каждым запросом серверу.
Единственное, что позволяется хранить в куках - это идентификатор сессии.
Под каждый номер (идентификатор) сессии, на самом сервере сохраняется файл - сериализованный массив.
И именно в этом файле записан пароль пользователя (или хешь) и конечно же имя пользователя.

Но этого нам конечно же не достаточно для более менее серьёзной безопасности. 
Ведь злоумышленник может украсть куку с идентификатором сессии жертвы и войти на сервер под его именем.

Для этого нам нужен пункт 4, а именно дополнительная защита, путем проверки ip и браузера, с которого входит потенциальный пользователь.


4) При успешной авторизации пользователя. Рассчитать переменную хэшь и поместить ее в массив $_SESSION
Код

$hesh = md5 ( ИП_клиента . информация_о_браузере клиента . специальное_кодовое_слово_которое_ придумал_автор_программы ) ;
$_SESSION['hesh'] = $hesh ;


При каждом запросе, если сессия ссылается на правильного пользователя, мы проверяем, а такой ли у него брузер и ip, который был при прошлом входе на сайт.

Для этого мы опять вычисляем хешь, и если он не совпадает с тем, который записан в $_SESSION['hesh'], то просим его ввести пароль и имя пользователя еще раз. 
Код

$hesh = md5 ( ИП_клиента . информация_о_браузере клиента . специальное_кодовое_слово_которое_ придумал_автор_программы ) ;
if ($_SESSION['hesh'] != $hesh)
{
    //Просим авторизоватся еще раз.
}


Это очень сильно повысит безопасность, по сравнению с обычный авторизацией через сессию, 
но тем не менее не обеспечит 100% безопасности, т.к. информацию, которая лётает по проводам, можно при очень огромном
желении перехватить.

5) Для этого остаётся последний метод борьбы - шифрование. Т.е. нужно организовать шифрование информации между браузером и сервером. Все серьёзные организации, связанные с деньгами делают это в обязательном порядке. Втроенный в браузеры алгоритм ширования называется SSL.


В заключение, еще раз повторю, что все это нужно только тогда, если речь идет о чем то действительно серьёзном. Иначе же, можно не парить мозг и спокойно ограничится только первыми тремя пунктами. 








Это сообщение отредактировал(а) NLspieler - 5.2.2010, 00:10
PM MAIL   Вверх
Simpliest
Дата 5.2.2010, 00:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3



Цитата(yngwie19 @  4.2.2010,  20:10 Найти цитируемый пост)
Подскажите пожалуйста не могли бы Вы дать ссылку на статью, где будет описан правильный

К сожалению не могу, поскольку никогда не занимался поиском таких статей.

В целом подход верный. Нужно просто не упускать таких казалось бы мелочей как:
mysql_real_escape_string()
хранение пароля в виде хеша md5, sha1, а не в открытом виде.
обязательной проверки логина на допустимые символы 
обязательной проверки, что мы таки получили все необходимые параметры, а не только была ли нажата кнопка.
и т.д.

Цитата(nginx @  4.2.2010,  21:54 Найти цитируемый пост)
пруф ор die();

пруф чего? smile примера выше недостаточно?
Ты только что получил значение из базы где логин был условием.
Ты надеешься что полученное значение будет отличаться от логина в условии запроса? smile Или как?
Пароль у тебя хранится в открытом виде что тоже не есть хорошо.
Абсолютно не проверяется, а есть ли что-то вообще в $_POST - что неправильно, в отличии кода того же топикстартера. 


--------------------
user posted image
PM   Вверх
NLspieler
Дата 5.2.2010, 00:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 619
Регистрация: 13.10.2008
Где: Берлин

Репутация: 16
Всего: 19



Прошу проверить мою писанину о безапасности на полноту и на дыры. 
Ведь тема действительно серьёзная и
не допускающая никаких холиваров, т.е. религиозных войн. 


PM MAIL   Вверх
Vasay
Дата 5.2.2010, 00:30 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2097
Регистрация: 8.3.2006

Репутация: 1
Всего: 73



NLspieler, 

Цитата

Под каждый номер (идентификатор) сессии, на самом сервере сохраняется файл - сериализованный массив.
И именно в этом файле записан пароль пользователя (или хешь) и конечно же имя пользователя.


Зачем куда либо писать пароль (или его хешь) ?

Цитата

Для этого нам нужен пункт 4, а именно дополнительная защита, путем проверки ip и браузера, с которого входит потенциальный пользователь.


ip может быть динамическим, идентификатор браузера подделать очень просто.   Так что ИМХО пункт 4 бессмысленный. 




А вообще - желательно не забывать про существование ООП и MVC. Ни того ни другого нет ни в одном примере упомянутом в этой теме.


--------------------
Придумать идеальную защиту от дурака невозможно, дураки, наудивление, изобретательны.
PM MAIL   Вверх
Simpliest
Дата 5.2.2010, 00:36 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3



Цитата(Vasay @  4.2.2010,  23:30 Найти цитируемый пост)
ip может быть динамическим, идентификатор браузера подделать очень просто.   Так что ИМХО пункт 4 бессмысленный

Смысл таки определенный есть. Это еще одна дополнительная помеха взломщику.

Цитата(Vasay @  4.2.2010,  23:30 Найти цитируемый пост)
А вообще - желательно не забывать про существование ООП и MVC. Ни того ни другого нет ни в одном примере упомянутом в этой теме. 

Эм?
Каким образом это относится к безопасности авторизации? smile



--------------------
user posted image
PM   Вверх
Vasay
Дата 5.2.2010, 00:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2097
Регистрация: 8.3.2006

Репутация: 1
Всего: 73



Цитата

Смысл таки определенный есть. Это еще одна дополнительная помеха взломщику.


При динамическом ip пользователя будет периодически выкидывать . Оно надо? 

Ну а информация о браузере не дает никакой доп защиты.


Цитата

Эм?
Каким образом это относится к безопасности авторизации? 

Разделяй и властвуй.  smile 

Все должно быть отдельно - работа с БД в одном месте, валидация в  другом, а отображение в третьем.  Это позволяет контролировать код, уменьшает его избыточность, позволяет разделять работу между разными людьми.

Как следствие, меньше ошибок - меньше уязвимостей.


--------------------
Придумать идеальную защиту от дурака невозможно, дураки, наудивление, изобретательны.
PM MAIL   Вверх
Simpliest
Дата 5.2.2010, 01:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3



Цитата(Vasay @  4.2.2010,  23:55 Найти цитируемый пост)
При динамическом ip пользователя будет периодически выкидывать . Оно надо? 

Это зависит от требований задачи.


Цитата(Vasay @  4.2.2010,  23:55 Найти цитируемый пост)
Разделяй и властвуй.   
....
Как следствие, меньше ошибок - меньше уязвимостей. 

Аааа, ну тогда могу порекомендовать еще подбирать хороших программистов smile они тоже делают меньше ошибок и меньше уязвимостей.
А так же не кодить с похмелья и т.д. smile

Но к теме топика это относится опосредованно.



--------------------
user posted image
PM   Вверх
Spiker
Дата 5.2.2010, 02:05 (ссылка)    | (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 265
Регистрация: 25.5.2005
Где: Спортзал

Репутация: -2
Всего: -2



Лучше что-бы выкидывало, чем пользователя взломали.))
______________

У меня следующий подход, как вам?
Код

function checkUser()
{
    if (!isset($_SESSION['user_id'])) {
        header('Location: ' . WEB_ROOT . 'index.php?view=singin');
        exit;
    }
    
    if (isset($_GET['logout'])) {
        doLogout();
    }
}


Код

function doLogin()
{

    $errorMessage = '';
    
    $userName = $_POST['txtUserName'];
    $password = $_POST['txtPassword'];
    
    if ($userName == '') {
        $errorMessage = 'You must enter your username';
    } else if ($password == '') {
        $errorMessage = 'You must enter the password';
    } else {
        $sql = "SELECT user_id
                FROM users 
                WHERE user_login = '$userName' AND user_password = PASSWORD('$password')"; 
        $result = mysql_query("SET NAMES cp1251") or die(mysql_error());
        $result = mysql_query("SET CHARACTER SET cp1251") or die(mysql_error());
        $result = dbQuery($sql);
    
        if (dbNumRows($result) == 1) {
            $row = dbFetchAssoc($result);
            $_SESSION['user_id'] = $row['user_id'];
            
            $sql = "INSERT INTO last_log (date, user_id) VALUES(NOW(), '{$row['user_id']}')";
            dbQuery($sql);
        
            if (isset($_SESSION['login_return_url'])) {
                echo "<script>document.location='". $_SESSION['login_return_url'] ."'</script>";
                exit;
            } else {
                echo "<script>document.location='". WEB_ROOT ."admin/index.php'</script>";
                exit;
            }
        } else {
            $errorMessage = 'Wrong username or password';
        }        
            
    }
    
    return $errorMessage;
}


Код

function doLogout()
{
    if (isset($_SESSION['user_id'])) {
        unset($_SESSION['user_id']);
        session_unregister('user_id');
    }
        
    header('Location:' . WEB_ROOT . 'index.php?view=signin');
    exit;
}



--------------------
Даваите жить дружно! (Леопольд.) :shy67:
PM MAIL MSN   Вверх
Vasay
Дата 5.2.2010, 02:13 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2097
Регистрация: 8.3.2006

Репутация: 1
Всего: 73



Simpliest, 

Цитата

Аааа, ну тогда могу порекомендовать еще подбирать хороших программистов smile они тоже делают меньше ошибок и меньше уязвимостей.
А так же не кодить с похмелья и т.д. smile

Но к теме топика это относится опосредованно.


Да нет - напрямую.  Человек учится, так пускай учится мыслить объектно, пускай учится разбивать приложение на слои.

Идеология должна быть примерно такой:

Есть сущность пользователя. Класс UserEntity cо свойствами id, userName, userHashPasswd, userEMail.....
Есть DAO в котором осуществляется вся работа с БД связанная с юзером. Класс UserDao с методами saveUser(), getUserById, getUserByName, getUserByNameAndPasswd.....

Есть классы контроллеров (контроллер для регистрации пользователя, контроллер для логина).

Могут быть командные объекты для представления POST запросов в виде объектов. Вспомогательные классы (валидаторы и тд....)

Могут быть менеджеры, но для начала можно и без них.

Добавлено через 7 минут и 14 секунд
Цитата

Лучше что-бы выкидывало, чем пользователя взломали.))


Ок. Давай посмотрим когда человек может получить id сессии. Тогда когда он либо имеет доступ к компьютеру, а соответственно и к его браузеру и к его ip.

Или когда он мониторит трафик. А соответственно в данном случае он может спокойно отмониторить и пароль. Особенно, если его придется вводить часто.

Защита тут только шифрование канала.




--------------------
Придумать идеальную защиту от дурака невозможно, дураки, наудивление, изобретательны.
PM MAIL   Вверх
Spiker
Дата 5.2.2010, 02:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 265
Регистрация: 25.5.2005
Где: Спортзал

Репутация: -2
Всего: -2



а что же тогда делать? надо все на сервере хранить.
Весь канал что-ли шифровать?

Это сообщение отредактировал(а) Spiker - 5.2.2010, 02:23


--------------------
Даваите жить дружно! (Леопольд.) :shy67:
PM MAIL MSN   Вверх
Simpliest
Дата 5.2.2010, 03:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3



Цитата(Vasay @  5.2.2010,  01:13 Найти цитируемый пост)
Да нет - напрямую.  Человек учится, так пускай учится мыслить объектно, пускай учится разбивать приложение на слои.

Я тебя понимаю, но так же понимаю, что все это скопом сразу не влазит в одну голову smile
Поэтому постепенность и последовательность.

Spiker, 
Уже писали, для особо тяжелых случаев используется SSL

Но в особо тяжелых случаях и это не панацея smile




--------------------
user posted image
PM   Вверх
NLspieler
Дата 5.2.2010, 03:33 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 619
Регистрация: 13.10.2008
Где: Берлин

Репутация: 16
Всего: 19



Spiker,
покажи или расскажи, что у тебя за сайт.
Скорее всего, твоего подхода к безопасности будет полностью достаточно. 

Ведь только последний помешанный идиот станет пытатся взламывать сайт анекдотов, что бы написать матершинный комментарий от имени другого пользователя.

Цитата

ip может быть динамическим, идентификатор браузера подделать очень просто.   Так что ИМХО пункт 4 бессмысленный


Ну слава богу, что только 4 пункт бессмысленный, 
хотя на самом деле он не такой и беесмысленный.

Проведем такой мысленный эксперимент.

Есть сервер, есть взломщик, есть жертва, которого пытается взломать этот взломщик.
Сразу обговорим, что php-код, который хранится на сервере взломщику не известен. И узнать он его не может.


Допустим, что взломщик удалось встроится в провод и он теперь может видеть, какой трафик проходит между сервером и потенцильаной жертвой.
Он обнаруживает, что жертва передаёт определенный id-сессии. 
Наш взломщик пытается авторизоватся на сайте при помощи этого идентификатора, 

но
сервер, обнаружив, что у него другой ip и даже другой браузер, и что хеши не сходятся, мирно просит его ввести логин и пароль.
короче, такой метод не взлома не действует. Пусть user_agent и можно подобрать, но как подобрать нужный ip?
Я, если честно, не знаю. И мне кажется, что это не возможно, кроме нескольких редких исключений.

Кроме того, взломщик же не знает, что ему нужно отослать одновременно правильный user_agent и правильный ip? 
Он же не знает исходный код.


Но конечно, же отсутствие пятого пункта (шифрования) практически полностью убивает пользу от описанного четвертого.


Но даже если есть даже шифрование, то дыры все равно остаются. 

Ведь если злоумышленник мониторит трафик, то он спокойно сможет подменить публичные пароли на свои. Если интересно, опишу этот процесс более подробнее.


Короче, делаю вывод, что никакой абсолютной безопасности не существует. Можно только достигнуть безопасность порядка 99,99%, но
стопроцентая не возможна впринципе. 








PM MAIL   Вверх
MoLeX
Дата 5.2.2010, 07:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Местный пингвин
****


Профиль
Группа: Модератор
Сообщений: 4076
Регистрация: 17.5.2007

Репутация: 46
Всего: 140



Цитата(nginx @  4.2.2010,  20:05 Найти цитируемый пост)
ЩИТО? Геноцид SHA1()

каждый использует то что ему роднее и привычные, разницы так таковой нету!


Цитата(nginx @  4.2.2010,  20:05 Найти цитируемый пост)
зачем? у него, как будто, результат другой будет?

хотя бы для того чтобы:
1. БД не продолжала искать по таблице, после того как нашла уже одну запись
2. при безграмотном кодинге бывает что на один и тот же логин принадлежит N пользователям, в этом случае его конструкция не пройдет
Код

if(mysql_num_rows($res)!=1)



Цитата(nginx @  4.2.2010,  20:05 Найти цитируемый пост)
см. мою тему:
http://forum.vingrad.ru/topic-283105/view-all.html

 smile  smile  smile 
два запроса!!!


yngwie19 если тебе не важна красота формы авторизации то можешь использовать вот это
Код

header('WWW-Authenticate: Basic realm="Auth"');
header('HTTP/1.0 401 Unauthorized');
// $_SERVER['PHP_AUTH_PW']
// $_SERVER['PHP_AUTH_USER']



--------------------
Amazing  smile 
PM MAIL WWW ICQ   Вверх
yngwie19
Дата 5.2.2010, 08:29 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 829
Регистрация: 15.6.2008
Где: Новгород

Репутация: нет
Всего: нет



Ребят спасибо Вам всем большое за то что столько времени уделили моему вопросу. Я думаю, что Я найду для себя правильный путь решения моей задачи. Единственное что бы Я хотел у Вас спросить это про использование SSL. Подскажите толковое руководство по нему. И вообще человек, который создал эту тему (т.е Я) сможет это реализовать? или тут нужны серьезные знания? Поскольку Я смотрю что все крупные сайты используют SSL.
PM MAIL   Вверх
Ипатьев
Дата 5.2.2010, 11:41 (ссылка) |    (голосов:5) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Завсегдатай
Сообщений: 2232
Регистрация: 5.7.2009

Репутация: 28
Всего: 37



Вообще, я бы начинал не с авторизации.
Ведь дыра в работе с SQL к собственно авторизации не относится. 
Сначала надо осваивать элементарные вещи, "кирпичики", из которых строится веб-приложение. Работа с базой, куками. Строкамии, почтой.
Авторизация - это уже здание, которое строится из этих кирпичиков. Если их не знать, ксли они будут пустые внутри - здание развалится. Даже если на форуме покажут чертежи супер-проекта.
PM MAIL   Вверх
Ответ в темуСоздание новой темы Создание опроса
Правила форума "PHP"
Aliance
IZ@TOP
skyboy
SamDark
MoLeX

Новичкам:

  • PHP редакторы собираются и обсуждаются здесь
  • Электронные книги по PHP, документацию можно найти здесь
  • Интерпретатор PHP, полную документацию можно скачать на PHP.NET

Важно:

  • Не брезгуйте пользоваться тегами [code=php]КОД[/code] для повышения читабельности текста/кода.
  • Перед созданием новой темы воспользуйтесь поиском и загляните в FAQ
  • Действия модераторов можно обсудить здесь

Внимание:

  • Темы "ищу скрипт", "подскажите скрипт" и т.п. будут переноситься в форум "Web-технологии"
  • Темы с именами: "Срочно", "помогите", "не знаю как делать" будут УДАЛЯТЬСЯ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, IZ@TOP, skyboy, SamDark, MoLeX, awers.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | PHP: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0750 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.