Как сделать пееадресацию на авторизацию?

Question

[kiril9011]

Здравтсвуйте! есть код закомментирован:

<?
/**
  * Скрипт проверки авторизации пользователей
  */

// Запускаем сессию, из которой будем извлекать логин и пароль
// авторизовавшихся пользователей
session_start();

 
// Подлючаем файл с пользовательскими функциями
require_once('functions.php');

/**
  * Чтобы определить авторизован ли пользователь, нам нужно 
  * проверить существуют ли в базе данных записи для его логина
  * и пароля. Для этого воспользуемся пользовательской функцией
  * проверки корректности данных авторизовавшегося пользователя.
  * Если эта функция вернет false, значит авторизации нет. 
  * При отсутствии авторизации мы просто переадресовываем 
  * пользователя к странице авторизации.
  */

// Если в сессии присуствуют данные и о логине, и о пароле,
// проверяем их
if(isset($_SESSION['login']) && $_SESSION['login'] && isset($_SESSION['password']) && $_SESSION['password']) {
	// Если проверка существующих данных завершается неудачей
	if(!checkAuth($_SESSION['login'], $_SESSION['password'])) {
		// Переадресовываем пользователя на страницу авторизации
		header('location: login.php');
		// Прекращаем выполнение скрипта
		exit;
	}
}
// Если данных либо о логине, либо о пароле пользователя нет,
// считаем что авторизации нет, переадресовываем пользователя
// на страницу авторизации
else {
	header('location: login.php');
	// Прекращаем выполнение сценария
	exit;
}
?>

я думаю вам понятно, мне надо если клиент не авторизован, его перекидывало на login.php, а код этот почему не работает, почему?

Comments

[freshik312]

А у тебя разве не идет (если проверка не верна - кидаем на авторизацию, в ином случает (т.е. прошел проверку) кидаем на авторизацию) ?

[ThunderCat]

каким боком тут тег фриланс? Вы это еще и продаете кому-то?

[ThunderCat]

// Если в сессии присуствуют данные и о логине, и о пароле,
// ищем разработчика этого скрипта, с целью отпинать по шарам, 
// проверяем их присутствие и если есть - отрываем нафиг

Answer 1

[Stalker_RED]

1. Вместо if(isset($foo) && $foo) можно проверять if (!empty($foo))
2. Пожалуйста, не записывайте пароль в сессию.
3. Заголовок Location заполнен неправильно.

Comments

[kiril9011]

Денис Ефремов, так как правильно изменить?

[Stalker_RED]

kiril9011, пользователь присылает логин + пароль
вы сразу проверяете их, и если все ок сохраняете в сессию user_id, может быть имя или логин, но БЕЗ ПАРОЛЯ.
Затем делаете переадресацию, если нужно.