Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [MySQL][PHP]Skrypt logowania
Kaarpiik
post
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
 
 Start new topic
Odpowiedzi (1 - 4)
Ramzaa
post
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 (IMG:style_emoticons/default/smile.gif)

Ten post edytował Ramzaa 24.08.2010, 20:22:23
Go to the top of the page
+Quote Post
Kaarpiik
post
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 (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Asmox
post
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 (IMG:style_emoticons/default/smile.gif) Który wszystko wytłumaczy
Go to the top of the page
+Quote Post
Kaarpiik
post
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
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

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

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

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