Login systém jak na to – PHP – Fórum – Programujte.com

Ahoj, jak správně a hlavně spolehlivě udělat login systém? Nyní to mám takhle to není moc spolehlivé:
User vyplní údaje
Vytvoří se mu session s id

Pokud zaškrt ltrvalé přihlášení do db se uloží hash a vytvoří se cookie s tímto hashem

Při vypršení session či zavření okna s eporovná cookie s hashem s db a pokud sedí, tak to logne automaticky.

Jak udělat opravdu spolehlivý login? Nebo máte někdo nějakou třídu o kterou by jste se chtěli podělit? :) Děkuji za odpovědi.

http://php.vrana.cz/…zivatelu.php

P.S. "login systém" je strašná, češtinu mrzačící patlanina. O přehnaném používání slova "systém" viz stejnojmennou kapitolu v http://www.dfens-cz.com/view.php?…

 Já to vždycky řeším takto:

if($_POST['submit']:
	if($_POST['username'] && $_POST['password']:
		$query=mysql_query('SELECT * FROM users WHERE username="'.$_POST['username'].'" AND password="'.$_POST['password'].'"');
		$result=mysql_fetch_array($query);
		if(empty($result['iduser'])):
			//Špatné jméno nebo heslo
		else:
			// Byl(a) jsi přihlášen.
			$_SESSION['logged']=$result['iduser'];
			print '<meta http-equiv="result" content="1; index.php">';
	else:
		// Musíte vyplnit všechny údaje
	endif;
endif;

#3 Yohaku
Ou, to vypadá nebezpečně. Kromě toho, že máš zvláštní styl zápisu, který snad ani nemůže fungovat (neukončovat závorky v podmínkách, zápis á la Python (nebo spíše Pascal?), nekontrolování existence $_POST pomocí isset...), je kód náchylný k SQL injection.

#4 yetty
V podmínkách závorky nemám, takže je ani nemůžu ukončovat. Python jsem nikdy neviděl, tak nevím... O "isset" vím, bývám ale líný to psát...  

#5 Yohaku
Otevírající závorky tam máš:

if($_POST['submit']:

Alespoň u mne to vyhazuje syntax error. Správně by mělo být:

if($_POST['submit']):

Kdyby to bylo jednou, tak to chápu jako překlep, ale máš to tam dvakrát za sebou.

Také ti tam jednou chybí endif.

Mimochodem, co tě vede psát styl dvojtečka-endif? V PHP se většinou píše C stylem:

if ($a == $b) {
	echo "Je to stejne!";
} else {
	echo "Je to jine.";
}

ja som si vytvoril triedu, s metodami, ktore vsetko pokryvaju;

konstruktor sa stara o nastavovanie properties prip. kontroluje $_COOKIE s hashom ak existuje, s hashom v db - ked prejde a overia sa aj dalsie veci ohladom relacie (to uz je na tebe ako to nastavis) tak sa nastavi aj property s user_id, prip. este predlzi platnost cookie pri trvalych loginoch ak o nejaky cas ma expirovat. Login a logout je uz klasika.

Vstavane sessions v PHP kvoli efektivite nepouzivam.

#7 Lordest

Vstavane sessions v PHP kvoli efektivite nepouzivam.

Jak je to myšlené? Co je na session neefektivní? 

#8 yetty
Je to z mojich skusenosti, ale samozrejme mozem sa aj mylit:

defaultny session handler v PHP pouziva pre storage normalne subory, co je u malych projektov jednoduche, ale u vacsich uz nepouzitelne (i ked samozrejme aj o databazovom pristupe sa da polemizovat, ale to uz musi byt velmi vytazene - v tom pripade treba memcached a pod.)

viem o session_set_save_handler() ale volat to pri kazdom nacitani nie je moc dobre. Aj keby sme mohli prenastavit server, tak sa stale vola session_start() a ten pracu (teraz hovorim cisto o db pristupe) znacne komplikuje - bud sa zbytocne robia zaznamy v databaze aj o anonymnych relaciach, ktore by sa mozno nikdy neprihlasili a tazko sa to cisti, alebo potom nastavaju problemy, ako overovat a kedy, prip. sa ani nevie, kedy cookie expiruje, ak sa o nej nevytvara nejaka dodatocna cookie s dalsou exp., alebo jej regeneracia pri logine, co neriesi predchadzajuce. v skratke, kopu zbytocnych pristupov k db, a veci okolo toho.

ked som si to uvedomil a vytvoril si uplne vlastny session handler, zredukoval som kod aj o 50%. princip je v tom, ze pokym sa uzivatel neprihlasi (resp. nema dane cookies), relacny system si ho vobec nevsima, zvysok som uz popisal.

samozrejme velmi zalezi na type projektu.

#6 yetty
Jo ty myslíš tyhle závorky. Toho jsem si ani nevšim :D Dvojitý překlep :D Styl "dvojtečka, endif" používám, protože se mi prostě víc líbí... :D Je to delší, ale nu což... :D