авторизация

[Visible]

Подскажите пожалуйста, как правильно расшифровать пароль и сделать авторизацию/регистрацию php по табличке account колонки username/sha_pass_hash. никак не выходит что только не перепробывал.

sha1($_POST['login'] ."',':','". $_POST['password'])

перепробывал различные варианты, но итог один

[SeT]

SELECT SHA1(CONCAT(UPPER(`username`), ':', UPPER(pass)));
Вы наверное забыли перевести логин и пароль в верхний регистр.

[Visible]

Перепробывал многие варианты, но в чем то ошибся..

if (!empty($_POST))
{
		$login = $db->quote($_POST['login']);
		$query = $db->prepare("SELECT `id` FROM `account` WHERE `username`=".$login." LIMIT 1");
		$query->execute();
		$rows = $query->fetchAll();

		if (count($rows) == 1)
		{
			$query = $db->prepare("SELECT `id` FROM `account` WHERE `username`=".$login." AND `sha_pass_hash`= SHA1(CONCAT(UPPER($_POST['login']), ':', UPPER($_POST['password']))) LIMIT 1");

			$query->execute();
			$row = $query->fetch(PDO::FETCH_ASSOC);
			$_SESSION['user_id'] = $row['id'];
			
			$time = 86400;
			
			if (isset($_POST['remember']))
			{
				setcookie('login', $login, time()+$time, "/");
				setcookie('password', $password, time()+$time, "/");
			}
			header('Location: index.php');
			exit;
		}
		else
		{
			echo "<meta http-equiv='refresh' content='0; url=/index.php'>";
		}
}

[Кот ДаWINчи]

ужасный код, хоть и через ПДО...
Опустим то, что твой код - это только кусок кода а не весь алгоритм. Надеюсь что $db где-то выше в начале файла объявляется.
далее ты просто выбираешь "SELECT `id` FROM `account` WHERE `username`=".$login." LIMIT 1"

и если такая запись существует снова сделать запрос уже с хешем пароля, и без проверки выполнения запроса тупо присваиваешь ИД-шник в сессию.

Такое конечно работать не будет.

[RomanRom2]

работать то будет, только зачем делать два запроса то? делайте один запрос сразу со всеми критериями.
и зачем же так изголяться с апперами сделайте просто один SHA1(UPPER( от строки
$_POST['login']) . ':' . $_POST['password']
смысл какой по отдельным кускам регистр переводить?

лично я люблю железобетонный код. совершенно тупой и без использования рюшек, не вызывающий никакого двоякого чувства. в частности SHA1(UPPER( я бы сделал на стороне пхп. это во первых проще и понятнее для будущей отладки, проще sql серверу и избавляет от передачи по сети собсно логина и пароля - это для фанатов ПДО, хотя тут на локалхосте это ПДО как пятое колесо.

[Кот ДаWINчи]

как говориться, "вынести за скобки" upper сам бог велел. И это даже не из-за железобетонности, а просто из соображений логики и оптимизации кода.

А вот на счет ПДО не согласен. в данный момент, когда неумолимо приближается пхп 6, писать проекты на библиотеке mysql просто убийственно. так как в 6-ке обещают что эта библиотека будет иметь статус "Deprecated", а преемница mysqli до сих пор очень сыра и нестабильна, то волей неволей приходится обращать свой взор на монструозный ПДО.

Надо будет в выходной выделить пару часиков и накидать примерный код движка с авторизацией. (да бы новички смогли понять всю логику взаимодействия пхп и базы)

Как сделаю - опубликую здесь.

[Visible]

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

[Visible]

вроде разобрался. сделал это так:

if (!empty($_POST))
{
		$login = $db->quote($_POST['login']);
		$pass = $db->quote(sha1(strtoupper($_POST['login'].":".$_POST['password'])));
		$query = $db->prepare("SELECT `id` FROM `account` WHERE `username`={$login} AND `sha_pass_hash`={$pass} LIMIT 1");
		$query->execute();
		$rows = $query->fetchAll();

		// если такой пользователь нашелся
		if (count($rows) == 1)
		{
			$query = $db->prepare("SELECT `id` FROM `account` WHERE `username`={$login} AND `sha_pass_hash`={$pass} LIMIT 1");
			$query->execute();
			$row = $query->fetch(PDO::FETCH_ASSOC);
			$_SESSION['user_id'] = $row['id'];
			
			$time = 86400;
			
			if (isset($_POST['remember']))
			{
				setcookie('login', $login, time()+$time, "/");
				setcookie('password', $password, time()+$time, "/");
			}
			
			header('Location: index.php');
			exit;

		}
		else
		{
			echo "<script>alert(\"Не правильно ввели логин/пароль.\");</script>";
			echo "<meta http-equiv='refresh' content='0; url=/index.php'>";
		}
}

Но есть вещичка, которую не пойму. почему теле условия недоступен результат запроса, приходится этот запрос дублировать. как будет правильно?
нашел выход через

$_SESSION['user_id'] = $rows['0']['id'];

[Кот ДаWINчи]

Вот накидал пример авторизации через сессии (без регистрации). Всё работает.

Сразу предупреждаю, что это пример, а не рабочий движок.