Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [MySQL][PHP]Skrypt logowania
Kaarpiik
post 24.08.2010, 19:54:36
Post #1





Grupa: Zarejestrowani
Postów: 59
Pomógł: 1
Dołączył: 20.01.2008
Skąd: Bełchatów

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

Witam prezentuje mój skrypt logowania:

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. ob_start();
  4. include('funkcje/funkcje_strona.php');
  5. include('funkcje/funkcje_baza.php');
  6. ?>
  7. <head>
  8. <meta http-equiv="Content-type" content="text/html; charset=iso-8859-2">
  9. <link rel="stylesheet" href="/style.css " type="text/css">
  10. </head>
  11.  
  12.  
  13. <?php
  14. if($_GET['login'] == 'check'){
  15.  
  16. $user_login = $_POST["user_login"];
  17. $user_password = $_POST["user_password"];
  18.  
  19. if($user_login == "" || $user_password == "") {
  20. func_wiadomosc('Błąd!', 'Nie wpisałeś loginu lub hasła!', '4', 'index.php');
  21. exit();
  22. }
  23.  
  24. else {
  25. db_connect();
  26. $query = "SELECT * FROM users WHERE user_login = '$user_login' AND user_password = '$user_password'";
  27. $result = mysql_query($query);
  28. $row = mysql_fetch_array($result);
  29. if($row['user_login'] !== $user_login && $row['user_password'] !== $user_password) {
  30. func_wiadomosc('Błąd!', 'Podane dane się nie zgadzają.', '4', 'index.php');
  31. exit();
  32. }
  33. else {
  34. $_SESSION['sesja'] = 'istnieje';
  35. $_SESSION['user_id'] = $row['user_id'];
  36. $_SESSION['user_login'] = $row['user_login'];
  37. $_SESSION['user_email'] = $row['user_email'];
  38. $_SESSION['user_name'] = $row['user_name'];
  39. $_SESSION['user_city'] = $row['user_city'];
  40. $_SESSION['user_gg'] = $row['user_gg'];
  41. $_SESSION['user_lastlogin'] = $row['user_lastlogin'];
  42. $_SESSION['user_joindate'] = $row['user_joindate'];
  43. $_SESSION['user_position'] = $row['user_position'];
  44.  
  45. func_wiadomosc('Witamy '.$_SESSION['user_login'].'!', 'Poprawnie zalogowano.', '4', 'index.php');
  46. }
  47. }
  48. }
  49.  
  50. if($_GET['login'] == 'wyloguj') {
  51. session_destroy();
  52. if($_SESSION['sesja'] != 'istnieje'){
  53. func_wiadomosc('Żegnaj '.$_SESSION['user_login'].'!', 'Poprawnie wylogowano!', '4', 'index.php');
  54. }
  55. }
  56. ?>
[PHP] pobierz, plaintext


Jest jedno ale. Skrypt ten działa z opóźnieniem tzn gdy próbuje się logować skrypt wykonuje wszystko poprawnie. Wyskakuje mi okienko poprawnie zalogowano i przenosi mnie po 4 sekundach do index.php, jednak ukazuje mi się nadal formularz logowania, gdy odświeże stronę wszystko jest ok zalogowało. Tak samo jest przy wylogowywaniu: klikam wyloguj, wyskakuje okienko wylogowało ale tak naprawdę nie wylogowało i z powrotem przenosi mnie do wylogowywania. Gdy spróbuje następnym razem wszystko jest ok. Tak jakby usuwało zmienne sesji po czasie i przy logowaniu tak jakby tworzyło zmienne sesji z małym opóźnieniem. Czym może to być spowodowane?
Go to the top of the page
+Quote Post
Ramzaa
post 24.08.2010, 20:20:02
Post #2





Grupa: Zarejestrowani
Postów: 207
Pomógł: 17
Dołączył: 4.08.2009

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

zacznijmy może od tego, że nie warto tworzyć tylu sesji, bo można stworzyć sesję "user_login", po czym funkcje do wyciągania danych użytkownika z bazy mysql. Jedno proste zapytanko i masz wszystkie informacje o danym użytkowniku. A tworzysz do najzwyczajniej w świecie: mysql_fetch_assoc(mysql_query("SELECT * FROM `uzytkownicy` WHERE `login` = '".$_SESSION['user_login']."' i zobacz jaki masz porządek od razu w kodzie. Następną rzeczą jest warunek sprawdzający sesję użytkownika czy jest zalogowany, czy nie. Nie rozumiem po co używasz if($_SESSION['sesja'] != 'istnieje'){, skoro możesz wcześniej zarejestrować sesję "zalogowany=true" i później sprawdzać warunek tak: if ($_SESSION['zalogowany'] {. Nie brzmi lepiej i przejrzyściej? Poza tym radzę poczytać o atakach sql_injection i xss, bo Twój skrypt jest bardzo na to podatny.

i proponuję poczytać jeszcze o funkcji isset, która prawdopodobnie rozwiąże Twój problem.

pozdrawiam smile.gif

Ten post edytował Ramzaa 24.08.2010, 20:22:23
Go to the top of the page
+Quote Post
Kaarpiik
post 24.08.2010, 22:07:37
Post #3





Grupa: Zarejestrowani
Postów: 59
Pomógł: 1
Dołączył: 20.01.2008
Skąd: Bełchatów

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

Dzięki za przydatne podpowiedzi. Może i racja z tymi zmiennymi, usunę niepotrzebne ale parę się przyda do szybkiego identyfikowania np user_lastlogin i user_position

Są jeszcze jakieś pomysły dlaczego ten konkretny skrypt nie działa? Wiem że może nie jest bezpieczny ale logować poprawnie powinien smile.gif
Go to the top of the page
+Quote Post
Asmox
post 24.08.2010, 22:18:39
Post #4





Grupa: Zarejestrowani
Postów: 359
Pomógł: 12
Dołączył: 16.01.2009

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

Kurde też miałem kiedyś taki numer, ale pamiętam to jak przez mgłę. Póki co prowizorycznie daj funkcję na przeładowanie strony, aż nie pojawi się jakiś stary wyjadacz smile.gif Który wszystko wytłumaczy


--------------------
Moja prywatna strona internetowa
Go to the top of the page
+Quote Post
Kaarpiik
post 26.08.2010, 09:00:04
Post #5





Grupa: Zarejestrowani
Postów: 59
Pomógł: 1
Dołączył: 20.01.2008
Skąd: Bełchatów

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

Sam do tego doszedłem czytając trochę o sesjach. Przy wylogowywaniu sprawdzam czy zmienna sesji 'sesja' ma wartość istnieje. Oczywiście że ma bo session_destroy(); nie usuwa zmiennych sesji. Trzeba użyć komendy unset.

@edit:

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. ob_start();
  4. include('funkcje/funkcje_strona.php');
  5. include('funkcje/funkcje_baza.php');
  6. ?>
  7. <head>
  8. <meta http-equiv="Content-type" content="text/html; charset=iso-8859-2">
  9. <link rel="stylesheet" href="/style.css " type="text/css">
  10. </head>
  11.  
  12.  
  13. <?php
  14. if($_GET['login'] == 'check'){
  15.  
  16. $user_login = $_POST["user_login"];
  17. $user_password = $_POST["user_password"];
  18.  
  19. if($user_login == "" || $user_password == "") {
  20. func_wiadomosc('Błąd!', 'Nie wpisałeś loginu lub hasła!', '4', 'index.php');
  21. exit();
  22. }
  23.  
  24. else {
  25. db_connect();
  26. $query = "SELECT * FROM users WHERE user_login = '$user_login' AND user_password = '$user_password'";
  27. $result = mysql_query($query);
  28. $row = mysql_fetch_array($result);
  29. if($row['user_login'] !== $user_login && $row['user_password'] !== $user_password) {
  30. func_wiadomosc('Błąd!', 'Podane dane się nie zgadzają.', '3', 'index.php');
  31. exit();
  32. }
  33. else {
  34. $_SESSION['zalogowany'] = 'true';
  35. $_SESSION['user_id'] = $row['user_id'];
  36. $_SESSION['user_login'] = $row['user_login'];
  37. $_SESSION['user_position'] = $row['user_position'];
  38.  
  39. func_wiadomosc('Witamy '.$_SESSION['user_login'].'!', 'Poprawnie zalogowano.', '4', 'index.php');
  40. exit();
  41. }
  42. }
  43. }
  44.  
  45. if($_GET['login'] == 'wyloguj') {
  46. unset($_SESSION['zalogowany']);
  47. unset($_SESSION['suser_id']);
  48. unset($_SESSION['suser_login']);
  49. unset($_SESSION['suser_position']);
  50. session_destroy();
  51. if(!$_SESSION['zalogowany']){
  52. func_wiadomosc('Żegnaj '.$_SESSION['user_login'].'!', 'Poprawnie wylogowano!', '4', 'index.php');
  53. exit();
  54. }
  55. }
  56. ?>
[PHP] pobierz, plaintext


Skrypt wygląda następująco. Problem z wylogowywaniem rozwiązany, a sprawa logowania wciąż pozostaje.

bump

Ten post edytował Kaarpiik 26.08.2010, 09:00:38
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 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 Wersja Lo-Fi Aktualny czas: 18.07.2025 - 12:20
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn