Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]Cookie a sesja
Forum PHP.pl > Forum > Przedszkole
krystian19
16.06.2011, 15:10:37
Witam. Mam taki prosty system rejestracji i logowania użytkowników z bazy danych na swojej stronce. Wszystko działa poprawnie. Chciałbym jednak zrobić coś na styl autologowania aby użytkownik za każdym razem nie musiał wpisywać loginu i hasła, a został albo automatycznie zalogowany albo zostało mu już login i haslo wyświetlone, gotowe tylko do zatwierdzenia. Kombinowałem na różne sposoby z setcookie($login, $haslo, time()+9999); if(!isSet($_COOKIE['wyslane']) && !isSet($_POST['wyslane'])){ ale nic mi z tego nie wychodzi :/ Bardzo proszę o pomoc. 

[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start(); // rozpoczecie sesji
  4. ?>
  5.  
  6.  
  7. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
  8. <title>stronka</title>
  9. <link rel="stylesheet" type="text/css" href="style.css" />
  10. <p><span class="styl2"><font face="Comic Sans MS" color="#006699">Zaloguj się</font></span><br>
  11. <div class="content2">
  12.  
  13. <?php
  14.  
  15. if (!isset($_SESSION['login'])) { 
  16.  
  17.     if ($_POST['wyslane']) { 
  18.  
  19.         include 'baza.php'; 
  20.         $tabela = 'uzytkownicy'; 
  21.  
  22.  
  23.         $login = $_POST["login"];
  24.         $haslo = $_POST["haslo"];
  25.  
  26. setcookie($login, $haslo, time()+9999);
  27.  
  28.  
  29.         $wynik=mysql_query("SELECT * FROM $tabela WHERE
  30.         login='$login' and haslo='$haslo' and status=0");
  31.  
  32.  
  33.         if (mysql_num_rows($wynik) == 1) {
  34.             $info = mysql_fetch_array($wynik);
  35.            echo '<span class="1">Nie aktywowałeś jeszcze konta</span>';
  36.  
  37.         }
  38.  
  39.  
  40.         $wynik=mysql_query("SELECT * FROM $tabela WHERE
  41.         login='$login' and haslo='$haslo' and status=1");
  42.  
  43.  
  44.         if (mysql_num_rows($wynik) == 1) {
  45.             $info = mysql_fetch_array($wynik);
  46.             $_SESSION["login"] = $info["login"];
  47. 			 $_SESSION["imie"] = $info["imie"];
  48. 			  $_SESSION["nazwisko"] = $info["nazwisko"];
  49. 			    $_SESSION["image"] = $info["image"];
  50.  
  51. 				$last_login = date('YmdHis');	
  52.  
  53.  
  54.  
  55.             header('Location: index.php ');
  56.         } else {
  57.             echo '<span class="2">Zostały wprowadzone złe dane!</span>';
  58.         }
  59.         mysql_close($polaczenie);
  60.     }
  61.  
  62.  
  63.     echo <<< KONIEC
  64.  
  65.     <form class="form" action="logowaniee.php" method="post">
  66.     <input type="hidden" name="wyslane" value="TRUE" />
  67.  
  68.     <p>
  69. 	  <div class="label"><label for="login"><font color="#000000">Login</font></label></div>
  70. <font color="#000000">
  71. 	  <input type="text" name="login" id="login" />
  72. </font>
  73.  
  74. 	<p>
  75. 	  <div class="label"><label for="haslo"><font color="#000000">Hasło</font></label></div>
  76. <font color="#000000">
  77. 	  <input type="password" name="haslo" id="haslo" />
  78. </font>
  79.  
  80.     <p class="submit2">
  81.       <font color="#000000">
  82.       <input type="submit" value="Zaloguj mnie" />
  83.       </font>
  84.     </p>
  85.  
  86. 	<p class="przypomnij">
  87. 		<a href="przypomnieniee.php"><font color="#000000">Nie pamietasz hasła?</font></a>
  88. 	</p>
  89.  
  90.     </form>
  91. KONIEC;
  92.  
  93. } else {
  94.     header('Location: index.php'); 
  95. }
  96.  
  97. if ($_GET["wylogowanie"] == "tak") {
  98.  
  99.     session_unset();
  100.     session_destroy();
  101.     header('Location: index.php'); 
  102. }
  103.  
  104. ?>
  105.  
  106.  
  107.  
  108. </div>
  109.  
  110. <p><br>
  111.  
  112.  
[PHP] pobierz, plaintext


Znalazłem na internecie taką pomoc, w podobnej sprawie:


do formularza dodałem:
<input type="checkbox" name="autologin" value="1">

a do pliku 
[PHP] pobierz, plaintext 
  1. <?php
  2. $suma_kontrolna=$_SESSION['login'].'_'.md5($_SESSION['login'].'!Q@W#E$R%T^Y&');
  3. if(isset($_POST['autologin'])) setcookie('autologin',$suma_kontrolna,time()+3600*24*14); // 14 dni
  4. ?>
[PHP] pobierz, plaintext 


[PHP] pobierz, plaintext 
  1. <?php
  2. if(isset($_COOKIE['autologin'])) {
  3.     $tab=explode('_',$_COOKIE['autologin']);
  4.     $suma_kontrolna=$tab[0].'_'.md5($tab[0].'!Q@W#E$R%T^Y&');
  5.     if($suma_kontrolna==$_COOKIE['autologin']) $_SESSION['login']=$tab[0]; // Automatyczne logowanie
  6. }
  7. ?>
[PHP] pobierz, plaintext


Po zalogowaniu (przy zaznaczeniu autologowania) - loguje się ok. Naciskam wyloguj. I teraz najważniejsze. Klikam w logowanie i automatycznie mnie loguje,czyli to o co mi chodziło, ale niestety nie jako dany użytkownik, tylko po prostu loguje.
Ponawiam prośbę o pomoc...

aachi
16.06.2011, 18:01:16
Nie bardzo wiem, gdzie u Ciebie jest problem. Podrzuć może aktualną wersję pliku, zamiast starą i informację, że gdzieś dodałeś jakiśtam kod.

Za to zauważyłem kilka innych rzeczy:
Tak troszkę dziwnie napisałeś ten swój skrypt. Niepotrzebnie wysyłasz dwa zapytania do bazy danych ( jedno ze status=0 i zaraz po nim to samo ze status=1). Nie lepiej wysłać jedno zapytanie i pobrać z wyniku ten status?

Zapisujesz w ciastku login i hasło (linia 26). Nie tylko jest to niebezpieczne, ale też zastanawiam się po co to robisz? Przecież masz je zapisane w bazie i nie widzę byś gdzieś to odczytywał później.

Wrzucasz też do zapytania sql dane bezpośrednio z $_POST. Dobrze jest użyć mysql_real_escape_string() na danych przesyłanych od użytkownika przed ich wrzuceniem do zapytania.

Wrzuć listing aktualnego kodu to może ktoś Ci pomoże.
krystian19
18.06.2011, 11:56:41
Witam, dzięki za rady AACHI. Ogólnie mój problem polega na tym, że chciałbym aby działało autologowanie. Nie wiem jak to zrobić prawidłowo, Kod który dorzuciłem ten z cookie po prostu skopiowałem. I to działa, niestety nie do końca. Mianowicie pierwszy raz loguje poprawnie, ale nastepnie już loguje ale nie jako dana osoba powiazana z sesją.
Zdaje sobie sprawę że zapisywanie ciasteczka jest niezbyt poprawne z uwagi na niebezpieczeństwo, ale nie mam innego pomysłu...
Podsumowując: - logowanie działa, chciałbym jednak, że za każdym razem jak dana osoba wejdzie na stronę będzie automatycznie zalogowana, aż do momentu kiedy naciśnie wyloguj.
Na chwile obecną jest zalogowana tylko na czas sesji. 



[PHP] pobierz, plaintext 
  1.  
  2. <?php
  3. ob_start();
  4. session_start(); // rozpoczecie sesji
  5. ?>
  6. <style type="text/css">
  7. <!--
  8. .styl2 {font-size: 18px}
  9. -->
  10. </style>
  11.  
  12. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
  13. <title>stronka</title>
  14. <link rel="stylesheet" type="text/css" href="style.css" />
  15. <p><span class="styl2"><font face="Comic Sans MS" color="#006699">Zaloguj się</font></span><br>
  16. <div class="content2">
  17.    1.
  18.       <?php
  19.       $suma_kontrolna=$_SESSION['login'].'_'.md5($_SESSION['login'].'!Q@W#E$R%T^Y&');
  20.       if(isset($_POST['autologin'])) setcookie('autologin',$suma_kontrolna,time()+3600*24*14); // DODANY KOD
  21.       ?>
  22.  
  23.  
  24.  
  25.  
  26. <?php
  27.  
  28.  
  29.  
  30. if (!isset($_SESSION['login'])) { 
  31.  
  32.     if ($_POST['wyslane']) { 
  33.  
  34.         include 'baza.php'; 
  35.         $tabela = 'uzytkownicy'; 
  36.  
  37.  
  38.         $login = $_POST["login"];
  39.         $haslo = $_POST["haslo"];
  40.  
  41.  
  42. if(isset($_COOKIE['autologin'])) {
  43.     $tab=explode('_',$_COOKIE['autologin']);
  44.     $suma_kontrolna=$tab[0].'_'.md5($tab[0].'!Q@W#E$R%T^Y&');
  45.     if($suma_kontrolna==$_COOKIE['autologin']) $_SESSION['login']=$tab[0]; // Automatyczne logowanie DODANY KOD
  46.  
  47. }
  48.  
  49.         $wynik=mysql_query("SELECT * FROM $tabela WHERE
  50.         login='$login' and haslo='$haslo' and status=0");
  51.  
  52.  
  53.         if (mysql_num_rows($wynik) == 1) {
  54.             $info = mysql_fetch_array($wynik);
  55.            echo '<span class="1">Nie aktywowałeś jeszcze konta</span>';
  56.  
  57.         }
  58.  
  59.  
  60.         $wynik=mysql_query("SELECT * FROM $tabela WHERE
  61.         login='$login' and haslo='$haslo' and status=1");
  62.  
  63.  
  64.         if (mysql_num_rows($wynik) == 1) {
  65.             $info = mysql_fetch_array($wynik);
  66.             $_SESSION["login"] = $info["login"];
  67. 			 $_SESSION["imie"] = $info["imie"];
  68. 			  $_SESSION["nazwisko"] = $info["nazwisko"];
  69. 			    $_SESSION["image"] = $info["image"];
  70.  
  71. 				$last_login = date('YmdHis');	
  72.  
  73.  
  74.  
  75.             header('Location: index.php ');
  76.         } else {
  77.             echo '<span class="2">Zostały wprowadzone złe dane!</span>';
  78.         }
  79.         mysql_close($polaczenie);
  80.     }
  81.  
  82.  
  83.     echo <<< KONIEC
  84.  
  85.     <form class="form" action="logowaniee.php" method="post">
  86.     <input type="hidden" name="wyslane" value="TRUE" />
  87.  
  88.     <p>
  89. 	  <div class="label"><label for="login"><font color="#000000">Login</font></label></div>
  90. <font color="#000000">
  91. 	  <input type="text" name="login" id="login" />
  92. </font>
  93.  
  94. 	<p>
  95. 	  <div class="label"><label for="haslo"><font color="#000000">Hasło</font></label></div>
  96. <font color="#000000">
  97. 	  <input type="password" name="haslo" id="haslo" />
  98. </font>
  99. <input type="checkbox" name="autologin" value="1">Autologowanie 
  100.     <p class="submit2">
  101.       <font color="#000000">
  102.       <input type="submit" value="Zaloguj mnie" />
  103.       </font>
  104.     </p>
  105.  
  106. 	<p class="przypomnij">
  107. 		<a href="przypomnieniee.php"><font color="#000000">Nie pamietasz hasła?</font></a>
  108. 	</p>
  109.  
  110.     </form>
  111. KONIEC;
  112.  
  113. } else {
  114.     header('Location: index.php'); 
  115. }
  116.  
  117. if ($_GET["wylogowanie"] == "tak") {
  118.  
  119.  
  120.     session_unset();
  121.     session_destroy();
  122.     setcookie('autologin',false);            //DODANY KOD
  123.     header('Location: index.php'); 
  124. }
  125.  
  126. ?>
  127.  
  128.  
  129.  
  130. </div>
  131.  
  132. <p><br>
  133.  
[PHP] pobierz, plaintext



aachi
19.06.2011, 14:57:26
Ja autologowanie robię tak, że jeśli ktoś zaznaczy podczas logowania, odpowiednią opcję, to zapisuję w bazie id użytkownika, przeglądarkę, czas wygaśnięcia autologowania i jakiś unikalny_ciąg. I ten unikalny ciąg zapisuję u użytkownika w ciastku.
Przy ponownym wejściu użytkownika (niezalogowanego oczywiście), sprawdzam czy istnieje odpowiednie ciastko z danymi. Jeśli istnieje to pobieram z bazy wiersz i sprawdzam czy dane się zgadzają. Jeśli nie wysyłam polecenie wygaśnięcia ciastka. Jeśli tak to ustawiam odpowiednie zmienne sesyjne.

Czyli w sumie robię to samo co i Ty próbujesz zrobić. smile.gif

Przeanalizowałem pobieżnie Twój kod i chyba znalazłem błąd...

Obecnie sprawdzasz czy ciastko "autologin" jest ustawione wyłącznie gdy $_POST['wyslane'] jest ustawione (a więc gdy ktoś skorzysta z formularza). No i jak pisałem... twój kod jest dziwny
Powinieneś mieć raczej:
[PHP] pobierz, plaintext 
  1. if (!isset($_SESSION['login'])) {   //Jeśli user nie jest zalogowany to
  2. if ($_POST['wyslane']) {          //Sprawdzamy czy user się loguje, jeśli tak to próbujemy go zalogować
  3.    include 'baza.php';
  4.    $tabela = 'uzytkownicy';
  5.    $login = mysql_real_escape_string($_POST["login"]);   //Zabezpieczamy się przed mysql injection
  6.    $haslo = mysql_real_escape_string($_POST["haslo"]);
  7.  
  8.    $wynik=mysql_query("SELECT * FROM $tabela WHERE login='$login' and haslo='$haslo' LIMIT 1") or die('Blad bazy!!!');   //Sprawdzamy czy istnieje osoba o takim loginie i hasle w bazie
  9.    if (mysql_num_rows($wynik) == 1) {     //Jeśli taka osoba istnieje to...
  10.       $info = mysql_fetch_array($wynik);
  11.       if ($info['status']==0) {            //Sprawdzamy czy osoba ma status 0. Jesli tak to wyświetlamy komunikat o potrzebie aktywacji
  12.       echo '<span class="1">Nie aktywowałeś jeszcze konta</span>';
  13.       } elseif ($info['status']==1) {     //Jeśli status=1 to logujemy osobę
  14.          $_SESSION["login"] = $info["login"];
  15.          $_SESSION["imie"] = $info["imie"];
  16.          $_SESSION["nazwisko"] = $info["nazwisko"];
  17.          $_SESSION["image"] = $info["image"];
  18.          $last_login = date('YmdHis');
  19.          header('Location: index.php ');
  20.       }
  21.    } else {     //Jeśli nie ma w bazie usera o podanym loginie i hasle, to wyświetlamy komunikat
  22.           echo '<span class="2">Zostały wprowadzone złe dane!</span>';
  23.    }
  24. } else {    //Jeśli user jest niezalogowany i nie skorzystał z formularza to sprawdzamy czy ma ustawione autologowanie
  25.    if(isset($_COOKIE['autologin'])) {   //Sprawdzamy czy istnieje ciastko
  26.       $tab=explode('_',$_COOKIE['autologin']);    
  27.       if (md5($tab[0].'!Q@WE$R%T^Y&')==$tab[1]) {  //Jeśli ciastko istnieje to sprawdzamy, czy dane w nim zapisane mają poprawny format
  28.       //Poniżej logujemy użytkownika
  29.          $login=mysql_real_escape_string($tab[0]);    //Zabezpieczamy przed sql injection dane przesłane od usera w ciastku
  30.          $wynik=mysql_query("SELECT * FROM $tabela WHERE login='$login' LIMIT 1") or die('Blad bazy!!!');   //Sprawdzamy czy istnieje osoba o takim loginie w bazie
  31.          if (mysql_num_rows($wynik) == 1) {     //Jeśli taka osoba istnieje to...
  32.             $info = mysql_fetch_array($wynik);
  33.             if ($info['status']==0) {            //Sprawdzamy czy osoba ma status 0. Jesli tak to wyświetlamy komunikat o potrzebie aktywacji
  34.                 echo '<span class="1">Nie aktywowałeś jeszcze konta</span>';
  35.             } elseif ($info['status']==1 {     //Jeśli status=1 to logujemy osobę
  36.             $_SESSION["login"] = $info["login"];
  37.             $_SESSION["imie"] = $info["imie"];
  38.             $_SESSION["nazwisko"] = $info["nazwisko"];
  39.             $_SESSION["image"] = $info["image"];
  40.             $last_login = date('YmdHis');
  41.             }
  42.          } else {     //Jeśli nie ma w bazie usera o podanym loginie,  to prawdopodobnie jest to próba włamania
  43.             setcookie('autologin','',1); // kasujemy ciastko
  44.             die();   //kończymy wykonywanie skryptu
  45.          }
  46.       }
  47.    }
  48. }
[PHP] pobierz, plaintext



To tak na szybko przerobiłem, bez sprawdzania czy nie ma błędów.... Przeanalizuj mój kod i swój i dokonaj odpowiednich poprawek.
Jeszcze raz zaznaczam, że mojego kodu nie uruchamiałem w PHP, więc nie wiem czy nie ma literówek lub innych błędów.
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.