Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [PHP+MySQL]Pomoc przy skrypcie
Lerno
post
Post #1





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

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 
  1. <?php
  2. session_start();
  3.    $zalogowany = &#092;"false\";
  4.    if(isset($_POST['login']))	  $login = $_POST['login'];
  5.    else					 $login = @$_SESSION['login'];
  6.    if(isset($_POST['pass']))	  $pass = $_POST['pass'];
  7.    else					 $pass = @$_SESSION['pass'];
  8.  
  9.    $db_info['host'] = &#092;"localhost\";
  10.    $db_info['user'] = &#092;"xxx\";
  11.    $db_info['pass'] = &#092;"xxx\";
  12.    $db_info['db']   = &#092;"xxx\";
  13.    $db_connection = mysql_pconnect($db_info['host'], $db_info['user'], $db_info['pass']) or die(mysql_error());
  14. 	  if(!$db_connection) $db_status = false;
  15.    $db_select = mysql_select_db($db_info['db'], $db_connection) or die(mysql_error());
  16. 	  if(!$db_select) $db_status = false;
  17.    
  18.    if((isset($_POST['login']))&amp;&amp;(isset($_POST['pass']))){	  
  19. 	  $login = addslashes($_POST['login']);
  20. 	  $pass = addslashes($_POST['pass']);
  21. 	  if(!isset($db_status) || $db_status !== false){
  22. 		 $query = mysql_query(&#092;"SELECT * FROM `user` WHERE `login`='\".$login.\"' AND `pass`='\".$pass.\"';\");
  23. 		 if(mysql_num_rows($query) == 1){
  24. 			$zalogowany = &#092;"true\";
  25. 			$_SESSION['login'] = $login;
  26. 			$_SESSION['pass'] = $pass;
  27. 			$wiersz = mysql_fetch_array($query);
  28. 			$_SESSION['name'] = $wiersz['name'];
  29. 			$_SESSION['email'] = $wiersz['email'];
  30. 			$user_logged = $wiersz['level'];
  31. 		 }
  32. 	  }   
  33.    }
  34.    /*
  35.    // wczytujemy liste userow z pliku oraz dokonujemy weryfikacji
  36.    if(file_exists('users.dat')){
  37. 	  $users_file = file('users.dat');
  38. 	  $users_amount = count($users_file);
  39. 		 // haszujemy haslo
  40. 	  $passmd5 = md5($pass);
  41.  
  42. 	  while(list($key, $val) = each($users_file)){
  43. 		 $user_line = explode('||', trim($val));
  44. 		 if(($user_line[1] == $login)&amp;&amp;($user_line[2] == $passmd5)){
  45. 			$zalogowany = \"true\";
  46. 			$_SESSION['login'] = $login;
  47. 			$_SESSION['pass'] = $pass;
  48. 			$user_logged = $user_line;
  49. 		 }
  50. 	  }
  51.    }
  52.    */
  53.    // jesli sie niezalogowal
  54.    if($zalogowany == &#092;"false\"){
  55. 	  echo '<dl><dt>Logowanie</dt><dt>•<a href=\"index.php?id=1\">Zaloguj</a></dt></dl>';
  56. 	  $text = '';
  57. 	  if(isset($_GET['id']) == &#092;"1\"){
  58. 		 // form logowania
  59. 		 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>';
  60. 	  }
  61.    }
  62.    // jesli sie zalogowal to
  63.    if($zalogowany == &#092;"true\"){
  64. 	  //opcja wylogowania
  65. 	  echo '<dl><dt><span>Wylogowanie</span></dt><dt><span>•</span> <a href=\"index.php?id=logout>Wyloguj</a></dt><dt>&amp;nbsp;</dt>';
  66. 	  if(@$_GET['id'] == 'logout'){
  67. 		 session_destroy();
  68. 		 echo 'Wylogowano użytkownika. W celu ponownego zalogowania kliknij <a href=\"index.php?id=1\" title=\"przelogowanie\">tutaj</a>';
  69. 	  }
  70. 	  // ustanawiamy poziom dostepu
  71. 	  if($user_logged == '1')		 $status = 'update';
  72. 	  else if($user_logged == '2')   $status = 'moderatorek';
  73. 	  else if($user_logged == '3')   $status = 'adminek';
  74. 	  else{
  75. 		 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>.';
  76. 		 @session_destroy();
  77. 	  }
  78.  
  79. 	  if($status == 'update'){
  80. 		 echo '<dt><span>Podstawowe</span></dt><dt><span>•</span> <a href=\"'. $_SERVER[&#092;"PHP_SELF\"] .'?id=1\">Dzial</a></dt>';
  81. 		 if(@$_GET['id'] == &#092;"1\"){
  82. 			echo 'Sztama!';
  83. 		 }
  84. 	  }
  85. 	  // itd...
  86.    }
  87. ?>
[PHP] pobierz, plaintext


Moze widzicie co robie zle. Z gory dzieki.


__________________
Moderated by Bakus: Wykorzystuj właściwe znaczniki BBCode
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #2





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

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 ?(IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Go to the top of the page
+Quote Post
Lerno
post
Post #3





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

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?
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #4





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

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 
  1. <?php
  2. if(zalogowany) {
  3.    switch($_GET['action']) {
  4. 		   case 'jaka_nazwa':
  5. 				   include('strona.inc.php');
  6. 				   break;
  7. 	}
  8. } else {
  9. 	echo &#092;"Niestety ale musisz sie zalogowac\";
  10. }
  11. ?>
[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 (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Pozdro (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Lerno
post
Post #5





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

(IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Nie chcialo ci sie spojrzec na skrypcik ktory wkleilem w 1 poscie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
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. :|
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #6





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

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 
  1. <?php
  2.  
  3. if($user_logged == '1')		 $status = 'update';
  4. 	  else if($user_logged == '2')   $status = 'moderatorek';
  5. 	  else if($user_logged == '3')   $status = 'adminek';
  6.  
  7. ?>
[PHP] pobierz, plaintext

tzn gdzie masz te zmienne ? z bazy je bierzesz?

Pozdro
Go to the top of the page
+Quote Post
Lerno
post
Post #7





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

Kod
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("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 
  1. <?php
  2.  
  3.    session_start();
  4.    $zalogowany = &#092;"false\";
  5.    if(isset($_POST['login']))	  $login = $_POST['login'];
  6.    else					 $login = @$_SESSION['login'];
  7.    if(isset($_POST['pass']))	  $pass = $_POST['pass'];
  8.    else					 $pass = @$_SESSION['pass'];
  9.  
  10. ?>
[PHP] pobierz, plaintext


Ten post edytował Lerno 11.02.2005, 11:52:04
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #8





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

No ok, i co dalej sie wychacza ? Jesli tak to napisz jak wpisujesz przejscie do innej strony...
Go to the top of the page
+Quote Post
Lerno
post
Post #9





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

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 
  1. <?
  2.  session_start();
  3. 	if(isset($_SESSION['name'])){
  4.   echo 'Zalogowany ' . $_SESSION['name'].'<br />';
  5.   echo '<p><a href=\"'.$_SERVER[&#092;"PHP_SELF\"].'?id=1\">menu 1</a><br />';
  6.   echo'<a href=\"'.$_SERVER[&#092;"PHP_SELF\"].'?id=logout\">Wyloguj</a></p>';
  7.  
  8.   if(@$_GET['id'] == &#092;"1\") echo 'Test';
  9.   if(@$_GET['id'] == &#092;"logout\"){
  10. 	  session_destroy();
  11. 	  echo 'Wylogowano';
  12.   };
  13. 	}
  14. 	else{
  15.   echo 'Niezalogowany';
  16.   //wyswietlamy formularz
  17.   echo '<form action=\"submit.php\" method=\"post\">
  18. 	Login: <input type=\"text\" name=\"login\" /><br />
  19. 	Hasło: <input type=\"password\" name=\"pass\" /><br />
  20. 	<input type=\"submit\" value=\"Zaloguj\" />
  21. 	  </form>
  22.   ';
  23. 	}
  24. ?>
  25. </body>
  26.  
  27. </html>
[PHP] pobierz, plaintext


submit.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. session_start();
  4. require('config.php'); 
  5. if((isset($_POST['login']))&&(!empty($_POST['login']))&&(isset($_POST['pass']))&&(!empty($_POST['pass']))){
  6. 	$login = $_POST['login'];
  7. 	$pass = $_POST['pass'];
  8. 	echo 'Dane jakie podales: ' . $login . ' i ' . $pass . '.<br />';
  9. 	$query = mysql_query(&#092;"SELECT * FROM `users` WHERE `login`='\".$login.\"' AND `pass`='\".$pass.\"';\");
  10. 	if(mysql_num_rows($query) == 1){
  11.   // przypisuje do sesji jego login i pass
  12.   $_SESSION['login'] = $login;
  13.   $_SESSION['pass'] = $pass;
  14.   // dodatkowo przypisuje inne dane dotyczace tego usera do sesji
  15.   $wiersz = mysql_fetch_array($query);
  16.   $_SESSION['name'] = $wiersz['name'];
  17.   $_SESSION['email'] = $wiersz['email'];
  18.   echo 'Zalogowales sie '.$_SESSION['name'] . ' o e-mail`u '. $_SESSION['email'].'<br />';
  19.   echo '<a href=\"index.php?id=1\">menu 1</a>';
  20. 	}
  21. 	else{
  22.   echo 'Nie ma takiego uzytkownika!';
  23. 	}
  24. }
  25. else{
  26. 	echo 'Nie podales danych!';
  27. }
  28.  
  29. ?>
[PHP] pobierz, plaintext


config.php
[PHP] pobierz, plaintext 
  1. <?
  2. $db_info['host'] = &#092;"localhost\";
  3. $db_info['user'] = &#092;"xxx\";
  4. $db_info['pass'] = &#092;"xxx\";
  5. $db_info['db']   = &#092;"xxx\";
  6. $db_connection = mysql_connect($db_info['host'], $db_info['user'], $db_info['pass']) or die('ble');
  7. $db_select = mysql_select_db($db_info['db'], $db_connection) or die('ble2');
  8. ?>
[PHP] pobierz, plaintext


Ten post edytował Lerno 11.02.2005, 17:01:22
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #10





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

No i dla mnie lajt (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bledow chyba nie ma (chodz moge sie mylic (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ). Wykorzystaj teraz to dobrze do przechodzenia przez strony i bedzie lajt (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Pozdro
Go to the top of the page
+Quote Post
Lerno
post
Post #11





Grupa: Zarejestrowani
Postów: 68
Pomógł: 0
Dołączył: 16.03.2004

Ostrzeżenie: (0%)
-----

Strzyzyk dzieki za helpa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Oczywiscie powinno byc jeszcze przed sprawdzaniem danych usera z info w bazie dodane addslashes(); To chyba uniemozliwi atak SQL injection, ale nie jestem pewien (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Mam nadzieje ze ktos z bardziej zaawansowanych ludzikow napisze tu cos wiecej o bezpieczenstwie powyzszego skryptu. Nadzieja matka glupich, ale co tam... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Jeszcze raz dzieki za helpa Strzyzyk.
Go to the top of the page
+Quote Post
Strzyzyk_MC
post
Post #12





Grupa: Zarejestrowani
Postów: 19
Pomógł: 0
Dołączył: 22.08.2003
Skąd: Bydgoszcz

Ostrzeżenie: (0%)
-----

N/p. Poza tym przed sprawdzeniem powinno byc chyba stripslashes a nie addslashes ?(IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Pozdr (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Go to the top of the page
+Quote Post
popbart
post
Post #13





Grupa: Zarejestrowani
Postów: 255
Pomógł: 0
Dołączył: 22.04.2004
Skąd: Żoliborz

Ostrzeżenie: (0%)
-----

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
Go to the top of the page
+Quote Post
« Następny starszy · Gotowe rozwiązania · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 23.08.2025 - 06:23
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn