Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP] Logowanie nie działa
Forum PHP.pl > Forum > Przedszkole
kezman
25.08.2009, 10:08:17
Cześć, piszę skrypt logowania i mam problem. Po kliknięciu w submit, strona tylko się odświeża - nie ma żadnych błędów, nie pisze, że login / hasło są nie poprawne, ani nie przenosi na podaną stronę. Proszę o pomoc.

[PHP] pobierz, plaintext 
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  2. <html xmlns="http://www.w3.org/1999/xhtml">
  3. <head>
  4. <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  5. <title>Logowanie</title>
  6. </head>
  7. <body>
  8.  
  9. <?
  10. error_reporting(E_ALL); 
  11. ini_set('display_errors',1);
  12.  
  13. require_once("config.php");
  14.  
  15. if(isset($_POST['submit'])){
  16.  
  17. $user = $_POST['user'];
  18.  
  19. $password = $_POST['password']; 
  20.  
  21. $salt = 'NS9Wvi5t74j4xXdNXHAFhkj89GV7UKgy2FI3hwXDS4YX4TJqiTMaO8snxoCNGRR';
  22.  
  23. $hashed = md5($password.$salt);
  24.  
  25. $sql = mysql_query("SELECT * FROM kezman WHERE user='$user' AND password='$hashed'"); 
  26.  
  27. $num_rows = mysql_num_rows($sql); 
  28.  
  29. if ($num_rows) { 
  30.  
  31. header("location:blabla.html");
  32.  
  33. }
  34. else
  35. {
  36. $msg="Podane dane są nie poprawne !";
  37. } 
  38. }
  39.  
  40. ?>
  41.  
  42.  
  43.  
  44.  
  45.  
  46.  
  47.  
  48. <!-- Formularz -->
  49.  
  50. <div id="login">
  51.  
  52.                 <form id="panel" name="panel" method="post" action="index.php">
  53.                 <input name="user" type="text" id="user" /><br />
  54.                 <input name="password" type="password" id="password" /><br />
  55.                 <input name="singin" type="submit" id="submit" value="Sign in !" />
  56.                 </form>
  57.  
  58. </div>
  59.  
  60. <!-- Koniec formularza -->
  61.  
  62.  
  63.  
  64. </body>
  65. </html>
[PHP] pobierz, plaintext
thek
25.08.2009, 10:19:08
Nazwa submita z tego co widzę powoduje błąd. Sprawdzasz
[PHP] pobierz, plaintext 
  1. if(isset($_POST['submit']))
[PHP] pobierz, plaintext
ale przecież Twój submit ma name="singin" i to jej powinieneś użyć. A tak masz sytuację gdy sprawdzenie submita w zmiennej POST wywala Ci FALSE zawsze bo nie ma takiej zmiennej smile.gif Sprawdź to sobie dodając print_r lub else za sprawdzeniem submita w stylu: "Nie naciskano submita" winksmiley.jpg Wywalać Ci będzie to pewnie nawet gdy go będziesz naciskał.

EDIT: I tak będziesz miał błędy bo przed headerem posłałeś Doctype( i wszystko aż do znacznika body) do przeglądarki już. Wrzucaj do dopiero po sprawdzaniu czy dostałeś POST. Swoją drogą zmiennej $msg nigdzie nie wyświetlasz winksmiley.jpg
kezman
25.08.2009, 10:41:55
Dodałem sesje, naprawiłem submit, ale dzieje się tak samo. Myślę, że coś z sesjami nagrzebałem :/ :

[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3. if(session_is_registered("done")){ 
  4.   header("location:blabla.html");
  5.  
  6.   ?>
  7. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  8. <html xmlns="http://www.w3.org/1999/xhtml">
  9. <head>
  10. <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  11. <title>Logowanie</title>
  12. </head>
  13. <body>
  14.  
  15. <?
  16. error_reporting(E_ALL); 
  17. ini_set('display_errors',1);
  18.  
  19. require_once("config.php");
  20.  
  21. if(isset($_POST['signin'])){
  22.  
  23. $user = $_POST['user'];
  24.  
  25. $password = $_POST['password']; 
  26.  
  27. $salt = 'NS9Wvi5t74j4xXdNXHAFhkj89GV7UKgy2FI3hwXDS4YX4TJqiTMaO8snxoCNGRR';
  28.  
  29. $hashed = md5($password.$salt);
  30.  
  31. $sql = mysql_query("SELECT * FROM kezman WHERE user='$user' AND password='$hashed'"); 
  32.  
  33. $num_rows = mysql_num_rows($sql); 
  34.  
  35. if ($num_rows) { 
  36. session_register("done");    
  37.  
  38.  
  39. }
  40. else
  41. {
  42. echo "Podane dane są nie poprawne !";
  43. } 
  44. }
  45. }
  46.  
  47. ?>
  48.  
  49.  
  50.  
  51.  
  52.  
  53.  
  54.  
  55. <!-- Formularz -->
  56.  
  57. <div id="login">
  58.  
  59.                 <form id="panel" name="panel" method="post" action="index.php">
  60.                 <input name="user" type="text" id="user" /><br />
  61.                 <input name="password" type="password" id="password" /><br />
  62.                 <input name="signin" type="submit" id="signin" value="Sign in !" />
  63.                 </form>
  64.  
  65. </div>
  66.  
  67. <!-- Koniec formularza -->
  68.  
  69.  
  70.  
  71. </body>
  72. </html>
[PHP] pobierz, plaintext
thek
25.08.2009, 12:08:03
Ustaw sobie wszystko po kolei. Sesję startujesz, robisz post, jeśli po poście "done" jest zarejestrowane to robisz redirecta, a jeśli nie to pójdzie dalej. Innymi słowy robisz sobie tak zwany diagram interakcji. Weź sobie logicznie na chłopski rozum rozpisz co się ma, kiedy i jak w skrypcie wykonywać czasowo. Wtedy zauważysz, że na logikę tak jest.
Startuje sesja dla pliku -> sprawdza się rejestracja przebiegła -> sprawdzasz post -> w zależności od post masz errory lub przekierowanie z ustawieniem na "done".
Tyle że sprawdzanie sesji wymagałoby dodatkowego jednego przebiegu więc musisz to przebudować.
Startuje sesja -> sprawdzasz post -> ustawiasz lub nie w sesji "done" -> jeśli "done" zarejestrowane to przekierowanie -> brak done oznacza błędy więc wyświetl je i formularz.
kezman
25.08.2009, 12:36:54
Bardzo dziękuję za to wytłumaczenie, na prawdę podbudowałeś mnie do ukończenia skryptu guitar.gif

Aktualnie wygląda to tak :

[PHP] pobierz, plaintext 
  1. <?
  2. session_start();
  3. ?>
  4. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  5. <html xmlns="http://www.w3.org/1999/xhtml">
  6. <head>
  7. <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  8. <title>Logowanie</title>
  9. </head>
  10. <body>
  11.  
  12. <?
  13. error_reporting(E_ALL); 
  14. ini_set('display_errors',1);
  15.  
  16. require_once("config.php");
  17.  
  18.  
  19.  
  20.  
  21.  
  22. if(isset($_POST['signin'])){
  23.  
  24. $user = mysql_escape_string($user);
  25.  
  26. $password = mysql_escape_string($password);
  27.  
  28. $salt = 'NS9Wvi5t74j4xXdNXHAFhkj89GV7UKgy2FI3hwXDS4YX4TJqiTMaO8snxoCNGRR';
  29.  
  30. $hashed = md5($password.$salt);
  31.  
  32. $sql = mysql_query("SELECT * FROM kezman WHERE user='$user' AND password='$hashed'"); 
  33.  
  34. $num_rows = mysql_num_rows($sql); 
  35.  
  36. if ($num_rows) { 
  37.  
  38.  
  39. session_register("user");
  40. echo "Działa !";
  41.  
  42. }
  43. else
  44. {
  45. echo "Podane dane są nie poprawne !";
  46. } 
  47. }
  48.  
  49.  
  50. ?>
  51.  
  52.  
  53.  
  54.  
  55.  
  56.  
  57.  
  58. <!-- Formularz -->
  59.  
  60. <div id="login">
  61.  
  62.                 <form id="panel" name="panel" method="post" action="index.php">
  63.                 <input name="user" type="text" id="user" /><br />
  64.                 <input name="password" type="password" id="password" /><br />
  65.                 <input name="signin" type="submit" id="signin" value="Sign in !" />
  66.                 </form>
  67.  
  68. </div>
  69.  
  70. <!-- Koniec formularza -->
  71.  
  72.  
  73.  
  74. </body>
  75. </html>
[PHP] pobierz, plaintext


Wszystko działa. Mam jednak pytanie. Czy skrypt jest bezpieczny/optymalny ? Jeśli nie to mógłbyś mi podać jakieś wskazówki ?
Dodałem coś takiego :

[PHP] pobierz, plaintext 
  1. $user = mysql_escape_string($user);
  2.  
  3. $password = mysql_escape_string($password);
[PHP] pobierz, plaintext
thek
25.08.2009, 12:59:04
Mysql_escape_string jest oznaczona jako deprecated, czyli ma być wycofana i zamiast niej powinno się używać mysql_real_escape_string, ale poza tym masz jeden "mały" błąd... Nie sprawdzasz danych przychodzących z formularza. Strip_tags lub funkcja adekwatnie działająca w takim wypadku to niezbędne minimum. W ten sposób zabezpieczasz się przed wrzuceniem kodu html do pól formularza. Poczytaj artykuły o Cross Site Scripting (CSS, XSS) i wstrzyknięciu kodu (SQL Injection) - czyli innymi słowy bezpieczeństwie formularzy. Tutaj w portalu masz o tym trochę. Na przyszłość radziłbym Ci także używać apostrofu pojedynczego zamiast podwójnego. Jest to szybsze, bo nie uruchamia interpretera PHP szukającego w stringu zmiennych php smile.gif Przykład?
[PHP] pobierz, plaintext 
  1. $sql = mysql_query("SELECT * FROM kezman WHERE user='$user' AND password='$hashed'"); 
[PHP] pobierz, plaintext

zamień na
[PHP] pobierz, plaintext 
  1. $sql = mysql_query('SELECT * FROM kezman WHERE user=\''.$user.'\' AND password=\''.$hashed.'\''); 
[PHP] pobierz, plaintext
Ważne by wiedzieć kiedy i jak chronić te apostrofy by były widziane odpowiednio. Stąd zauważ, że czasem muszę je "escapować". W sumie zastanawiam się czy de facto musze, ponieważ mysql_real_escape powinna otoczyć string apostrofami sama winksmiley.jpg Ale nigdy nie testowałem tongue.gif
kezman
25.08.2009, 13:36:27
Bardzo Ci dziękuję ! Pozdrawiam
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.