Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP]System logowania php - dobry ?
Forum PHP.pl > Forum > Przedszkole
revi
8.11.2016, 20:50:44
Napisałem sobie skrypt logowania do strony.
I mam takie pytanie: Czy to jest dobre zabezpieczenie questionmark.gif (hasło oczywiście będzie inne wink.gif )


INDEX:

[PHP] pobierz, plaintext 
  1. <?php  session_start(); 
  2.  
  3.  
  4.  
  5. ?>
  6. <html>
  7. <head>
  8. <title>Strona na haslo</title>
  9. </head>
  10. <body>
  11. <form method="post" action="">
  12.         Hasło <input value="" type="password" name="haslo" />
  13.         <input type="submit" name="submit" value="Zaloguj" />
  14. </form>
  15.  
  16. <a href="tajne.php">ZASZYFROWANA STRONA</a>
  17.  
  18.  
  19.  
  20. <?php
  21. error_reporting(E_ALL ^ E_NOTICE);
  22. $haslo = $_POST['haslo'];
  23. $zakodowanehaslo = password_hash("test", PASSWORD_DEFAULT);
  24.  
  25.  
  26.  
  27.  
  28. if(password_verify($haslo, $zakodowanehaslo)) 
  29. {
  30.  
  31.  $_SESSION["test1"] = "zalogowany"; 
  32.  header("Refresh: 0; tajne.php"); 
  33. }else{
  34. 	header("/");
  35. }
  36.  
  37. ?>
  38.  
  39.  
  40. </body>
  41. </html>
  42.  
[PHP] pobierz, plaintext








TAJNE.PHP:



[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. error_reporting(E_ALL ^ E_NOTICE);
  4. if ($_SESSION["test1"] == "zalogowany")
  5. {
  6. ?>
  7. <html>
  8. <head>
  9. 	<title>Tajne</title>
  10. </head>
  11. <body>
  12. Witaj<br>
  13. <a href="logout.php"><button>WYLOGUJ</button></a><br><br><br><br>
  14.  
  15.  
  16. LAJ LAJL AJ
  17. gsdgsdgadgs
  18. sdgsadgsad
  19. ążźćżźć
  20.  
  21.  
  22. </body>
  23. </html>
  24.  
  25.  
  26. <?php
  27.  
  28. }else{
  29.  
  30.  header("Refresh: 0; /stronanahaslo");
  31. }
  32.  
  33. ?>
[PHP] pobierz, plaintext
nospor
8.11.2016, 20:57:22
eeee.... masz jedno haslo dla wszystkich userow. To raz
A dwa nie ma co tego hasla kodowac skoro i tak wszystko to jedno haslo zapisane w pliku

ps:
error_reporting(E_ALL ^ E_NOTICE);
masz reportowac wszystkie bledy nawet NOTICE bo inaczej pewnego dnia obudzisz sie z reka w nocniku
revi
8.11.2016, 21:02:27
znaczy stronka przeznaczona tylko dla mnie, tak żeby nikt inny nie miał do niej dostępu, nie będzie żadnego innego użytkownika.

/edit


error_reporting(E_ALL ^ E_NOTICE); - usunięte
hasło poprawione.


Poza tym wszystko ok ?
nospor
8.11.2016, 21:05:16
Punkt nr 2 i 3 nadal aktualny. Tak jak to zrobiles, kodowanie hasla nie ma zadnego sensu
revi
8.11.2016, 21:08:05
INDEX:

[PHP] pobierz, plaintext 
  1. <?php  session_start(); 
  2.  
  3.  
  4.  
  5. ?>
  6. <html>
  7. <head>
  8. <title>Strona na haslo</title>
  9. </head>
  10. <body>
  11. <form method="post" action="">
  12.         Hasło <input value="" type="password" name="haslo" />
  13.         <input type="submit" name="submit" value="Zaloguj" />
  14. </form>
  15.  
  16. <a href="tajne.php">ZASZYFROWANA STRONA</a>
  17.  
  18.  
  19.  
  20. <?php
  21. $haslo = $_POST['haslo'];
  22.  
  23. if($haslo == "test") 
  24. {
  25.  
  26.  $_SESSION["test1"] = "zalogowany"; 
  27.  header("Refresh: 0; tajne.php"); 
  28. }else{
  29. 	header("/");
  30. }
  31.  
  32. ?>
  33.  
  34.  
  35. </body>
  36. </html>
  37.  
[PHP] pobierz, plaintext








TAJNE.PHP 


[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. if ($_SESSION["test1"] == "zalogowany")
  5. {
  6. ?>
  7. <html>
  8. <head>
  9. 	<title>Tajne</title>
  10. </head>
  11. <body>
  12. Witaj<br>
  13. <a href="logout.php"><button>WYLOGUJ</button></a><br><br><br><br>
  14.  
  15.  
  16. LAJ LAJL AJ
  17. gsdgsdgadgs
  18. sdgsadgsad
  19. ążźćżźć
  20.  
  21.  
  22. </body>
  23. </html>
  24.  
  25.  
  26. <?php
  27.  
  28. }else{
  29.  
  30.  header("Refresh: 0; /stronanahaslo");
  31. }
  32.  
  33. ?>
[PHP] pobierz, plaintext
Shou
8.11.2016, 22:04:29
To typowy przykład logowania. OK, pod warunkiem, że hasło będzie wystarczająco trudne. W przeciwnym razie warto dodać jakikolwiek mechanizm antywłamaniowy zliczający błędnie wpisane dane.
nospor
9.11.2016, 10:23:35
Ja ci nie kazalem kasowac tej linii
error_reporting(E_ALL ^ E_NOTICE);

Ja mowilem o jej modyfikacji czyli
error_reporting(E_ALL);

No chyba ze w php.ini okresliles poziom logowania bledow
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.