Forum PHP.pl > [PHP+MySQL]Pomoc przy skrypcie

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP+MySQL]Pomoc przy skrypcie

Forum PHP.pl > Forum > Gotowe rozwiązania

Lerno

10.02.2005, 16:59:22

Od dluzszego czasu pisalem skrypty w php oparte na plikach, dzialaly dobrze i sie sprawdzaly. Teraz jednak chce liznac bazy danych np: mysql. Napisalem jakis czas temu logowanie i chcialbym teraz je przerobic na mysql. Stworzylem baze i tabele w mysqlu i teraz pisze pod nia skrypt. Niedziala. Tzn. logowanie dziala niby, ale po tym jak sie zaloguje jakbym gubil sesje, albo raczej polaczenie z baza, czy... cos tam innego sie psuje. W kazdym razie logowanie przebiega poprawnie, ale po zalogowaniu czego bym nie zrozobil zwraca mi ze jestem niezalogowany i koniec. Moze trzeba zapisywac w bazie stan? Na plikach dzialalo mi w takiej postaci, na mysqlu juz nie.

Oto skrypt (miedzy /**/ umiescilem tez kod ktory pobiera dane z pliku i uzywajac tej weryfikacji zamiast bazy skrypt bedzie dzialal poprawnie.)

[PHP] pobierz, plaintext 
<?php
session_start();
   $zalogowany = &#092;"false\";
   if(isset($_POST['login']))	  $login = $_POST['login'];
   else					 $login = @$_SESSION['login'];
   if(isset($_POST['pass']))	  $pass = $_POST['pass'];
   else					 $pass = @$_SESSION['pass'];
 
   $db_info['host'] = &#092;"localhost\";
   $db_info['user'] = &#092;"xxx\";
   $db_info['pass'] = &#092;"xxx\";
   $db_info['db']   = &#092;"xxx\";
   $db_connection = mysql_pconnect($db_info['host'], $db_info['user'], $db_info['pass']) or die(mysql_error());
	  if(!$db_connection) $db_status = false;
   $db_select = mysql_select_db($db_info['db'], $db_connection) or die(mysql_error());
	  if(!$db_select) $db_status = false;
   
   if((isset($_POST['login']))&amp;&amp;(isset($_POST['pass']))){	  
	  $login = addslashes($_POST['login']);
	  $pass = addslashes($_POST['pass']);
	  if(!isset($db_status) || $db_status !== false){
		 $query = mysql_query(&#092;"SELECT * FROM `user` WHERE `login`='\".$login.\"' AND `pass`='\".$pass.\"';\");
		 if(mysql_num_rows($query) == 1){
			$zalogowany = &#092;"true\";
			$_SESSION['login'] = $login;
			$_SESSION['pass'] = $pass;
			$wiersz = mysql_fetch_array($query);
			$_SESSION['name'] = $wiersz['name'];
			$_SESSION['email'] = $wiersz['email'];
			$user_logged = $wiersz['level'];
		 }
	  }   
   }
   /*
   // wczytujemy liste userow z pliku oraz dokonujemy weryfikacji
   if(file_exists('users.dat')){
	  $users_file = file('users.dat');
	  $users_amount = count($users_file);
		 // haszujemy haslo
	  $passmd5 = md5($pass);
 
	  while(list($key, $val) = each($users_file)){
		 $user_line = explode('||', trim($val));
		 if(($user_line[1] == $login)&amp;&amp;($user_line[2] == $passmd5)){
			$zalogowany = \"true\";
			$_SESSION['login'] = $login;
			$_SESSION['pass'] = $pass;
			$user_logged = $user_line;
		 }
	  }
   }
   */
   // jesli sie niezalogowal
   if($zalogowany == &#092;"false\"){
	  echo '<dl><dt>Logowanie</dt><dt>•<a href=\"index.php?id=1\">Zaloguj</a></dt></dl>';
	  $text = '';
	  if(isset($_GET['id']) == &#092;"1\"){
		 // form logowania
		 echo'<div >Zaloguj się.</div><div ><form action=\"index.php\" method=\"post\"><div>&amp;nbsp;</div><div >Login:<br /> <input class=\"pola_input\" type=\"password\" name=\"login\" value=\"\" /></div><div >Hasło:<br /> <input class=\"pola_input\" type=\"password\" name=\"pass\" value=\"\" /><br /><input type=\"submit\" value=\"Zaloguj\" /></div></form></div>';
	  }
   }
   // jesli sie zalogowal to
   if($zalogowany == &#092;"true\"){
	  //opcja wylogowania
	  echo '<dl><dt><span>Wylogowanie</span></dt><dt><span>•</span> <a href=\"index.php?id=logout>Wyloguj</a></dt><dt>&amp;nbsp;</dt>';
	  if(@$_GET['id'] == 'logout'){
		 session_destroy();
		 echo 'Wylogowano użytkownika. W celu ponownego zalogowania kliknij <a href=\"index.php?id=1\" title=\"przelogowanie\">tutaj</a>';
	  }
	  // ustanawiamy poziom dostepu
	  if($user_logged == '1')		 $status = 'update';
	  else if($user_logged == '2')   $status = 'moderatorek';
	  else if($user_logged == '3')   $status = 'adminek';
	  else{
		 echo 'Zostałeś/łaś wylogowany/a! W celu próby ponownego zalogowania się kliknij <a href=\"adm.php?id=1\" title=\"Zaloguj się\">tutaj</a>.';
		 @session_destroy();
	  }
 
	  if($status == 'update'){
		 echo '<dt><span>Podstawowe</span></dt><dt><span>•</span> <a href=\"'. $_SERVER[&#092;"PHP_SELF\"] .'?id=1\">Dzial</a></dt>';
		 if(@$_GET['id'] == &#092;"1\"){
			echo 'Sztama!';
		 }
	  }
	  // itd...
   }
?>
[PHP] pobierz, plaintext

Moze widzicie co robie zle. Z gory dzieki.

__________________
Moderated by Bakus: Wykorzystuj właściwe znaczniki BBCode

Strzyzyk_MC

11.02.2005, 00:08:27

Wiesz co.... Bardzo probowalem sie skupic ale skrypt jak dla mnie malo czytelny i strasznie zamotany.... Nie lepiej bedzie zrobic od nowa dla logowania na db ?

A tak na logike... Jesli Ci "gubi" logowanie to zobacz czy poprawnie zostaja zapisane zmienne w sesjach.. Jesli sa dobrze zapisywane to wiadomo ze cos z ich odczytem jest zle ... Wszystko niby na chlopski rozum a moze wiele pomoc

Lerno

11.02.2005, 00:51:07

Od nowa chetnie bym napisal, ale wlasnie nie wiem za bardzo w czym tkwi problem.
Bo tu sie loguje, podejrzewam ze chodzi bardziej o polaczenie z baza, niz sesje. Chociaz... kurcze nie wiem.

Skrypt poprostu:
-sprawdza czy sa dane wpisane z formularza, lub czy istnieje sesja.
-potem polaczenie z baza
-jak sa zmienne to sprawdza czy baza znajdzie wynik odpowiedni
-jak znajdzie to przypisuje dane do sesji i dalej to juz wiadomo...

Po zalogowaniu jednak kazde przejscie na strone inna to zmienna ustawiona po skutecznym logowaniu na true zmienia sie na false ($zalogowany).

Moze ktos inny wie gdzie tkwi blad?

Strzyzyk_MC

11.02.2005, 01:04:45

No to moze zrob tak:
Nie wiem jak przechodzisz do kolejnych stronek ale zrob w ten sposob: index.php?action=jaka_nazwa i teraz w tym samym pliku dodaj np. cos takiego:

[PHP] pobierz, plaintext 
<?php
if(zalogowany) {
   switch($_GET['action']) {
		   case 'jaka_nazwa':
				   include('strona.inc.php');
				   break;
	}
} else {
	echo &#092;"Niestety ale musisz sie zalogowac\";
}
?>
[PHP] pobierz, plaintext

Chodzi mi o to ze poprostu to czy ktos jest zalogoany czy nie, nie wychodzi poza glowny plik czyli np. index.php
Jesli zrobisz cos takiego i nie bedzie dzialac tzn. ze zle tworzysz sejse, albo zle porownujesz...

Acha i jeszcze jedno

Samo logowanie zrob np. w ten sposob.... Akrypt pobiera login i haslo, nastepnie jest wykonywane zapytanie do bazy typu: "SELECT login FROM tabela WHERE login='$login' AND pass=password('$pass')"
i w tej chwili jak otrzymasz zapytanie zwroci Ci login tzn. ze sie zgadza i wtedy robisz np. takie cos:
$_SESSION['login'] = $login (ten login co otrzymales z bazy danych)
i teraz w skrypcie ktory napisalem wyzej w funkcji if gdzie napisalem zalogowany wpisujesz np. tak:
if(isset($_SESSION['login'])) {
.....
}

Mam nadzieje ze chociaz troszke mnie zrozumiales:P Godzina pozna wiec moze troszke niejasno napisalem, niemniej wydaje mi sie ze troche mozna z tego wyniesc

Pozdro

Lerno

11.02.2005, 10:26:12

Nie chcialo ci sie spojrzec na skrypcik ktory wkleilem w 1 poscie

Mam 2 pliki:
index.php -> pelni role szablonu i includuje plik (register.php)
register.php -> powyzszy kod, w ktorym:

Jesli logowanie przebieglo pomyslnie to oznacza ze zmienna $zalogowany = true;

Kod

if($zalogowany == "true"){

Po zalogowaniu daje opcje przejscia do podstrony panelu:

Kod

echo '<dt><span>Podstawowe</span></dt><dt><span>•</span> <a href="'. $_SERVER["PHP_SELF"] .'?id=1">Dzial</a></dt>';
         if(@$_GET['id'] == "1"){
            echo 'Sztama!';
         }

Wiec niby jest to o czym pisales. :|

Strzyzyk_MC

11.02.2005, 10:43:20

So... Zamiast przypisywac do zmiennej zalogowany true i false proponuje zrobic tak jak mowilem tzn. zeby zapytanie zwracalo login i jak zwroci to przypisac je do sesji i sprawdzac czy sesje istnieje... W koncu to jest zmienna superglobalna i nie bedzie trzeba sie martwic na przyszlosc...
Po 2 nie wiem skad to sie wzielo w ogole:

[PHP] pobierz, plaintext 
<?php
 
if($user_logged == '1')		 $status = 'update';
	  else if($user_logged == '2')   $status = 'moderatorek';
	  else if($user_logged == '3')   $status = 'adminek';
 
?>
[PHP] pobierz, plaintext

tzn gdzie masz te zmienne ? z bazy je bierzesz?

Pozdro

Lerno

11.02.2005, 11:14:37

Kod

if((isset($_POST['login']))&&(isset($_POST['pass']))){
      $login = addslashes($_POST['login']);
      $pass = addslashes($_POST['pass']);
      if(!isset($db_status) || $db_status !== false){
         $query = mysql_query("SELECT * FROM `user` WHERE `login`='".$login."' AND `pass`='".$pass."';");
         if(mysql_num_rows($query) == 1){
            $zalogowany = "true";
            $_SESSION['login'] = $login;
            $_SESSION['pass'] = $pass;
            $wiersz = mysql_fetch_array($query);
            $_SESSION['name'] = $wiersz['name'];
            $_SESSION['email'] = $wiersz['email'];
            $user_logged = $wiersz['level'];
         }
      }
   }

Jak widzisz tutaj do $user_logged po zalogowaniu przypisuje dane z bazy. Te dane to numerki ktore oznaczaja poziom dostepu/uprawnienia. Dalej przypisuje tym numerka juz poprostu wartosci aby np zamiast "2" wyswietlic "Administrator" i po to jest $status. Potem tez w zaleznosci od tego jaki ma gosc status dostaje takie opcje uzytkownika jakie sie mu naleza.

A co do logowania to na poczatku sprawdzam czy jest zmienna loginu i hasla tzn wypelnil formularz logowania, jesli nie to sprawdz czy istnieje sesja.

[PHP] pobierz, plaintext 
<?php
 
   session_start();
   $zalogowany = &#092;"false\";
   if(isset($_POST['login']))	  $login = $_POST['login'];
   else					 $login = @$_SESSION['login'];
   if(isset($_POST['pass']))	  $pass = $_POST['pass'];
   else					 $pass = @$_SESSION['pass'];
 
?>
[PHP] pobierz, plaintext

Strzyzyk_MC

11.02.2005, 14:54:25

No ok, i co dalej sie wychacza ? Jesli tak to napisz jak wpisujesz przejscie do innej strony...

Lerno

11.02.2005, 16:59:17

Napisalem od nowa, starajac sie analizowac krok po kroku no i chyba to dalo efekt, bo zdaje sie ze dziala. Chcialbym sie teraz zapytac czy ta metoda jaka ponizej zamieszcze jest BEZPIECZNA! Prosze o wszelkie opinie, nawet jakies "polslowka", najwyzej poszukam i zglebie temat. Chcialbym poprostu miec pewnosc, czy nie mozna tego byle jakim trickiem obejsc ot tak.

index.php

[PHP] pobierz, plaintext 
<?
 session_start();
	if(isset($_SESSION['name'])){
  echo 'Zalogowany ' . $_SESSION['name'].'<br />';
  echo '<p><a href=\"'.$_SERVER[&#092;"PHP_SELF\"].'?id=1\">menu 1</a><br />';
  echo'<a href=\"'.$_SERVER[&#092;"PHP_SELF\"].'?id=logout\">Wyloguj</a></p>';
 
  if(@$_GET['id'] == &#092;"1\") echo 'Test';
  if(@$_GET['id'] == &#092;"logout\"){
	  session_destroy();
	  echo 'Wylogowano';
  };
	}
	else{
  echo 'Niezalogowany';
  //wyswietlamy formularz
  echo '<form action=\"submit.php\" method=\"post\">
	Login: <input type=\"text\" name=\"login\" /><br />
	Hasło: <input type=\"password\" name=\"pass\" /><br />
	<input type=\"submit\" value=\"Zaloguj\" />
	  </form>
  ';
	}
?>
</body>
 
</html>
[PHP] pobierz, plaintext

submit.php

[PHP] pobierz, plaintext 
<?php
 
session_start();
require('config.php'); 
if((isset($_POST['login']))&&(!empty($_POST['login']))&&(isset($_POST['pass']))&&(!empty($_POST['pass']))){
	$login = $_POST['login'];
	$pass = $_POST['pass'];
	echo 'Dane jakie podales: ' . $login . ' i ' . $pass . '.<br />';
	$query = mysql_query(&#092;"SELECT * FROM `users` WHERE `login`='\".$login.\"' AND `pass`='\".$pass.\"';\");
	if(mysql_num_rows($query) == 1){
  // przypisuje do sesji jego login i pass
  $_SESSION['login'] = $login;
  $_SESSION['pass'] = $pass;
  // dodatkowo przypisuje inne dane dotyczace tego usera do sesji
  $wiersz = mysql_fetch_array($query);
  $_SESSION['name'] = $wiersz['name'];
  $_SESSION['email'] = $wiersz['email'];
  echo 'Zalogowales sie '.$_SESSION['name'] . ' o e-mail`u '. $_SESSION['email'].'<br />';
  echo '<a href=\"index.php?id=1\">menu 1</a>';
	}
	else{
  echo 'Nie ma takiego uzytkownika!';
	}
}
else{
	echo 'Nie podales danych!';
}
 
?>
[PHP] pobierz, plaintext

config.php

[PHP] pobierz, plaintext 
<?
$db_info['host'] = &#092;"localhost\";
$db_info['user'] = &#092;"xxx\";
$db_info['pass'] = &#092;"xxx\";
$db_info['db']   = &#092;"xxx\";
$db_connection = mysql_connect($db_info['host'], $db_info['user'], $db_info['pass']) or die('ble');
$db_select = mysql_select_db($db_info['db'], $db_connection) or die('ble2');
?>
[PHP] pobierz, plaintext

Strzyzyk_MC

11.02.2005, 17:09:50

No i dla mnie lajt

Bledow chyba nie ma (chodz moge sie mylic

). Wykorzystaj teraz to dobrze do przechodzenia przez strony i bedzie lajt

Pozdro

Lerno

11.02.2005, 17:46:03

Strzyzyk dzieki za helpa

Oczywiscie powinno byc jeszcze przed sprawdzaniem danych usera z info w bazie dodane addslashes(); To chyba uniemozliwi atak SQL injection, ale nie jestem pewien

Mam nadzieje ze ktos z bardziej zaawansowanych ludzikow napisze tu cos wiecej o bezpieczenstwie powyzszego skryptu. Nadzieja matka glupich, ale co tam...

Jeszcze raz dzieki za helpa Strzyzyk.

Strzyzyk_MC

12.02.2005, 00:19:09

N/p. Poza tym przed sprawdzeniem powinno byc chyba stripslashes a nie addslashes ?

Pozdr

popbart

12.02.2005, 02:45:09

Cytat

Oczywiscie powinno byc jeszcze przed sprawdzaniem danych usera z info w bazie dodane addslashes(); To chyba uniemozliwi atak SQL injection, ale nie jestem pewien winksmiley.jpg

Polecam mysql_escape_string" title="Zobacz w manualu PHP" target="_manual

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.