Witam !

Ostatnio pisałem sobie skrypt obsługi logowania. Obsługuje on system rang, domyślnie przyznając każdemu poziom user'a. Potem przez panel administracyjny admin może zmienić rangę użytkownikowi. Zależy mi na waszych sugestiach oraz ewentualnych wskazówkach. W planach mam jeszcze mechanizm zapamiętywania użytkownika za pomocą cookies. Oto kod:

[PHP] pobierz, plaintext 
<?
include ('functions.php');
include ('config.inc');
require('sesja.php');
 
db($serwer,$user,$pass,$baza);
ob_start();
if (isset($_POST['loginsend']))
{
 
	$login=htmlspecialchars(trim($_POST['login']));
	$check_login=mysql_query("SELECT login FROM `user_test` WHERE login='".$login."'");
	$login_result=mysql_fetch_array($check_login);
			if ($login_result['login'] = 1)
			{
				$login=htmlspecialchars(trim($_POST['login']));
				$pass=md5(md5($_POST['pass']));
				$check_user=mysql_query("SELECT count(*) FROM `user_test` WHERE login='".$login."' AND pass='".$pass."';");
				$if_ok=mysql_result($check_user, 0);
				if ($if_ok == 1)
				{
				$session_data="SELECT * FROM `user_test` WHERE login='".$login."'";
				$result=mysql_fetch_array(mysql_query($session_data));
				session_register('login');
				$_SESSION['login']=$_POST['login'];
				session_register('level');
				$_SESSION['level']=$result['level'];
				echo 'Zalogowano pomyslnie - nastapi przeladowanie strony...';
				$cookie=$_POST['save'];
				  if ($cookie = 1)
				  {
					 $dane=$login.'|'.$pass;
					 setcookie ("dane", $dane,time()+3600*24*365);
				  }
				}
			}
			else
			print "Nie ma takiego użytkonika !";
 
 
 
 
 
}else
include ('login.php');
 
db_close();
ob_end_flush();
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$pass=md5(md5($_POST['pass']));
?>
[PHP] pobierz, plaintext 

A czemu tak?

Tak mi się wydawało dobrze...Lepiej to wrzucić w zapytanie SQL ?

chodzil mi czemu przepuszczasz to dwa razy przez md5() ?

Żeby podwójnie zakodować..tak dla bezpieczeństwa....wiem że MD5 jest bardzo trudny do odkodowania ale md5 chyba nie obciąża serwera więc dlaczego nie ?

Zmieniłem kod....mianowicie dodał obsługę zapamiętywanie usera przez cookies....zapraszam do komentowania i ewentualnych komentarzy ze wskazówkami...

po co dajesz komunikat o poprawnym zalogowaniu nie lepiej odrazu przeladowac header .... np do ustawien konta

częśc postow zostal rozdzielona i przeniesona do tego tematu:
http://forum.php.pl/index.php?act=ST&f=1&t...t=0#entry236076