[php] sesja i md5 Opcje

[atomp3]

Witam

Mam dziwny problem napisalem klase ktora obejmuje zarzadzanie uzytkowniakmi. Uzywam kodowania md5 i base64 oto kawlaek kodu odpowiedzilany za sprawdzenie uzytkownika:

[PHP] pobierz, plaintext 
<?php
function checkUser()
	{
		$user = $this -> fetchData($this->table,'userName','userName = "'.base64_encode($_POST[login]).'" ',false,true);
		$pswd = $this -> fetchData($this->table,'userPass','userPass = "'.md5($_POST[password]).'" ',false,true);
 
 
			if ($user) {
				if ($pswd) {
					return true;
				}
				return false;
			}
 
 
	}
?>
[PHP] pobierz, plaintext 

Nastepnie zaleznie od wyniku ustawia sesje. Po wylogowaniu unset sesja. dziala to dobrze tylko ze jakims spsobem ustawilem jako pierwsze haslo: 123 co ciekawe po zmianie hasla przez uzytkownikow haslo 123 dziala nadal. Nie wiem gdzie i skad skoro w bazie jest tylko 1 wpis, sesja zostaje usunieta po wylogowaniu szukam wszedzie i nie wiem dlaczego haslo 123 nadal dziala

Ten post edytował atomp3 6.08.2007, 09:47:18

[kossa]

odpowiedź może być tu: fetchData - wykonujesz taką funkcję, a co tam siędzieje to już nie wiadomo bo nie ma kodu

druga sprawa tak na szybko to funkcja checkUser() ma zwracać true lub false, a jeśli nie spełniony jest warunek if ($user) to nie masz obsługi tego i nie zwróci flase bo nie wejdzie już do kontroli if ($paswd) gdzie dop. jest false, więc tu jest błąd logiczny

powinno być tak

[PHP] pobierz, plaintext 
<?php
if ($user) {
//kod dot. hasla
			}
else
return false;
?>
[PHP] pobierz, plaintext 

Łukasz

[atomp3]

Faktycznie moje pytanie nie jest poprawnie sformulowane ale musialbym wtedy caly kod wkleic. Funkcja fetchData zwraca pojedynczy wynik z bazy. Blad logiczny poprawilem. Generalnie funkcja dziala, ale gdzies pamieta to 123

[empathon]

Przyjrzyj się swojemu kodowi, zadziała dla każdego hasła istniejącego w bazie.
U mnie np. logowanie wygląda tak:

[PHP] pobierz, plaintext 
<?php
$c = new Criteria();
		$c->add(UserPeer::LOGIN_NAME, $login);
		$user = UserPeer::doSelectOne($c);
 
		//login exists
		if($user)
		{
			//check password
			if(sha1($user->getSalt().$password) == $user->getPassword())
			{
				$this->getContext()->getUser()->signIn($user, $password);
				return true;
			}
		}
?>
[PHP] pobierz, plaintext 

[atomp3]

Faktycznie totalna glupote napisalem anyway poprawilem i dziala dzieki za pomoc