Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]proste logowanie
Forum PHP.pl > Forum > Przedszkole
Lukasz_mk2
24.11.2009, 15:24:50
Witam, mam poblem z prostym logowaniem i nie wiem gdzie leży przyczyna. Generalnie skrypt łączy się z bazą użytkownik i hasło (hasło w bazie oczywiście kodowane md5) jest poprawne ale nie przechodzi od strony main.php
index.php
[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3. session_destroy();
  4.     $message=""; // Wiadomość po wylogowaniu
  5.  
  6. // Sekcja logowania
  7. $login=$_POST['login'];
  8. if($login) {
  9.   // W momencie kliknięcia w przycisk formularza
  10.   $uzytkownik=$_POST['uzytkownik'];
  11.   $md5_haslo=md5($_POST['haslo']);
  12.   // Koduje hasło funkcją md5().
  13.  
  14.  
  15. // Połączenie z bazą.
  16.   $host="localhost"; // Nazwa hosta.
  17.   $db_user="root"; // Nazwa użytkownika - MySQL.
  18.   $db_password="vertrigo"; // Hasło do bazy.
  19.   $database="baza"; // Nazwa bazy.
  20.     mysql_connect($host,$db_user,$db_password);
  21.     mysql_select_db($database);
  22. // Sprawdzenie nazwy użtkownika i hasła.
  23.   $rezultat=mysql_query("select * FROM logowanie WHERE uzytkownik='$uzytkownik' AND haslo='$md5_haslo'");
  24.     if(@mysql_num_rows($rezultat)){
  25.    session_register("uzytkownik"); // Zapamiętuje zmienną sesji
  26.     header("location:main.php"); // Przekierowanie do strony main.php
  27.   exit;
  28.   }else {
  29.   $message="Nieprawidłowa nazwa użytkownika lub hasło";
  30.   }
  31. } // Koniec sprawdzania autoryzacji.
  32. ?>
  33. <head>
  34.  
  35. <title>Dokument</title>
  36. </head>
  37.   <body>
  38. <? echo $message; ?>
  39. <form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">
  40.   <table>
  41.     <tr>
  42.       <td>Użytkownik: </td>
  43.       <td><input name="uzytkownik" type="text" id="uzytkownik" /></td>
  44.     </tr>
  45.     <tr>
  46.       <td>Hasło: </td>
  47.       <td><input name="haslo" type="password" id="haslo" /></td>
  48.     </tr>
  49.   </table>
  50. <input name="login" type="submit" id="login" value="login" />
  51. </form>
  52. </body>
  53. </html>
[PHP] pobierz, plaintext


main.php
[PHP] pobierz, plaintext 
  1. <?
  2.  
  3. session_start(); // Na samej górze zadeklaruj sesję.
  4.   if(!session_is_registered($uzytkownik)){ // Sprawdza zmienną sesji.
  5.   header("location:index.php"); // Przekierowanie do index.php
  6. }
  7. ?>
  8.  
  9. <head>
  10.  
  11. <title>Dokument</title>
  12. </head>
  13.     <body>
  14. <p>Witaj <? echo $_SESSION["uzytkownik"]; ?>! Jesteś zalogowany.</p>
  15. <p><a href="index.php">Wyloguj</a></p>
  16. </body>
  17. </html>
[PHP] pobierz, plaintext
darko
24.11.2009, 15:40:17 
[PHP] pobierz, plaintext 
  1. if(!session_is_registered("uzytkownik")){ // Sprawdza zmienną sesji.
[PHP] pobierz, plaintext
Lukasz_mk2
25.11.2009, 00:23:18
dzięki, działa

zdaje sobie sprawę że ten system logowania nie jest najlepszy ale czy można go jakoś przerobić żeby "rozróżniał użytkowników" tzn że jak wpisze innego użytkownika (istniejącego w bazie) to przekieruje go na inną stronę a nie tylko main.php
darko
25.11.2009, 00:29:45
Wszystko da radę, zacznij od przeprojektowania bazy, tak żeby mieć w tabeli users pole role (uprawnienie). I ustal sobie że admin ma role=0, user ma role=1 ktoś jeszcze niżej role=2 itd. Przy sprawdzaniu loginu i hasła wyciągnij sobie poziom uprawnień i wstaw chociażby w switch i case, gdziesz będzie "inkludował" odpowiednią stronę dla danych uprawnień. To oczywiście będzie działać, ale eleganckie, wydajne, elastyczne i superbezpieczne nie jest.

Acha! I obowiązkowo jeśli już zweryfikujesz usera, że to ten, za którego się podaje, to zapisz jego id i poziom uprawnień w sesji, żeby móc pominąć sprawdzanie loginu i hasła na każdej kolejnej podstronie którą mu udostępnisz.

ps. zrób przynajmniej podstawową walidację danych przysłanych z formularza, poczytaj o htmlspecialchars, strip_tags, htmlentities albo korzystaj z PDO, preparuj i binduj zapytania przed wykonaniem.

Pozdrawiam i powodzenia w pisaniu!
Lukasz_mk2
25.11.2009, 00:49:15
zrobiłem coś takiego
[PHP] pobierz, plaintext 
  1. <?
  2. // Na samej górze zadeklaruj sesję.
  3. session_start();
  4. // Sekcja wylogowywania, kasuje wszystkie zmienne sesyjne
  5. session_destroy();
  6.     $message=""; // Wiadomo?ć po wylogowaniu
  7.  
  8. // Sekcja logowania
  9. $login=$_POST['login'];
  10. if($login) {
  11.   // W momencie kliknięcia w przycisk formularza
  12.   $uzytkownik=$_POST['uzytkownik'];
  13.   $md5_haslo=md5($_POST['haslo']);
  14.   // Koduje hasło funkcj? md5().
  15.  
  16.  
  17.  
  18. // Poł?czenie z baz?.
  19.   $host="localhost"; // Nazwa hosta.
  20.   $db_user="root"; // Nazwa użytkownika - MySQL.
  21.   $db_password="vertrigo"; // Hasło do bazy.
  22.   $database="sklep"; // Nazwa bazy.
  23.     mysql_connect($host,$db_user,$db_password);
  24.     mysql_select_db($database);
  25. // Sprawdzenie nazwy użtkownika i hasła.
  26.  if ($rezultat=mysql_query("select * FROM logowanie WHERE uzytkownik='$uzytkownik' AND haslo='$md5_haslo' AND upr=0 "));
  27.     if(@mysql_num_rows($rezultat)){
  28.    session_register("uzytkownik");   // Zapamiętuje zmienn? sesji
  29.     header("location:main.php"); // Przekierowanie do strony main.php
  30.   exit;
  31.   }else {
  32.   $message="Nieprawidłowa nazwa użytkownika lub hasło";
  33.   }
  34.   if ($rezultat=mysql_query("select * FROM logowanie WHERE uzytkownik='$uzytkownik' AND haslo='$md5_haslo' AND upr=1 "));
  35.     if(@mysql_num_rows($rezultat)){
  36.    session_register("uzytkownik");   // Zapamiętuje zmienn? sesji
  37.     header("location:admin.php"); // Przekierowanie do strony admin.php
  38.   exit;
  39.   }else {
  40.   $message="Nieprawidłowa nazwa użytkownika lub hasło";
  41.   }
  42.  
  43. } // Koniec sprawdzania autoryzacji.
  44. ?>
  45. <head>
  46.  
  47. <title>Dokument</title>
  48. </head>
  49.   <body>
  50. <? echo $message; ?>
  51. <form id="form1" name="form1" method="post" action="<? echo $PHP_SELF; ?>">
  52.   <table>
  53.     <tr>
  54.       <td>Użytkownik: </td>
  55.       <td><input name="uzytkownik" type="text" id="uzytkownik" /></td>
  56.     </tr>
  57.     <tr>
  58.       <td>Hasło: </td>
  59.       <td><input name="haslo" type="password" id="haslo" /></td>
  60.     </tr>
  61.   </table>
  62. <input name="login" type="submit" id="login" value="login" />
  63. </form>
  64. </body>
  65. </html>
[PHP] pobierz, plaintext

po zalogowaniu jako użytkownik z upr=0 jest ok, ale przy użytkowniku z upr 1 przechodzi na admin.php ale wyskakuje komunikat z błędem sesji
plik admin.php
[PHP] pobierz, plaintext 
  1. <?
  2. session_start(); // Na samej górze zadeklaruj sesję.
  3.   if(!session_is_registered("uzytkownik")){ // Sprawdza zmienn? sesji.
  4.   header("location:index.php"); // Przekierowanie do index.php
  5. }
  6. ?>
  7. <html>cos</html>
[PHP] pobierz, plaintext
darko
25.11.2009, 00:58:56
Popracuj nad tym

$uzytkownik=$_POST['uzytkownik'];

wystarczy w formularzu logowania wpisać w pole user coś takiego: ' or 1=1 /*

i jesteś zalogowany
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.