Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [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
 
 Start new topic
Odpowiedzi
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

Posty w temacie
- Kaarpiik   [MySQL][PHP]Skrypt logowania   24.08.2010, 19:54:36
- - Ramzaa   zacznijmy może od tego, że nie warto tworzyć tylu ...   24.08.2010, 20:20:02
- - Kaarpiik   Dzięki za przydatne podpowiedzi. Może i racja z ty...   24.08.2010, 22:07:37
- - Asmox   Kurde też miałem kiedyś taki numer, ale pamiętam t...   24.08.2010, 22:18:39
- - Kaarpiik   Sam do tego doszedłem czytając trochę o sesjach. P...   26.08.2010, 09:00:04

« 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: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

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

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