Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> Skrypt logowania, czy jest on wystarczająco bezpieczny???
qba10
post
Post #1





Grupa: Zarejestrowani
Postów: 264
Pomógł: 20
Dołączył: 19.08.2007
Skąd: Bryńsk k. Lidzbarka Welskiego (nie mylić z Warmińskim)

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

[PHP] pobierz, plaintext 
  1. <?php
  2. if( isset($_POST['pamietaj']) AND $_POST['pamietaj'] == "ok") { 
  3.   setcookie("JumpPHP_".md5('login'),md5(strtolower($_POST['login'])) ,time()+60*60*24*30);
  4.   setcookie("JumpPHP_".md5('haslo'),md5($_POST['haslo']) ,time()+60*60*24*30);
  5. 	}	
  6. 	if(eregi("config.php",$_SERVER['PHP_SELF']) OR eregi("config.php",$_SERVER['QUERY_STRING'])) 
  7.  { header("Location: /"); die(); }
  8.  
  9.  
  10.  
  11. $_CONFIG['MySQL']['host'] = "allalalal"; 
  12. $_CONFIG['MySQL']['user'] = "bebebebbe"; 
  13. $_CONFIG['MySQL']['pass'] = "zapomniałem"; 
  14. $_CONFIG['MySQL']['base'] = "fajna"; 
  15.  
  16. if(!isset($_SESSION['sesja'])) { 
  17.  session_start(); 
  18.  $_SESSION['sesja'] = ''; 
  19. }
  20.  
  21.  
  22. @mysql_connect($_CONFIG['MySQL']['host'], $_CONFIG['MySQL']['user'], $_CONFIG['MySQL']['pass']) or die('<html> 
  23. <head> 
  24.  <meta http-equiv="Content-type" content="text/html; charset=iso-8859-2"> 
  25. <title>Error - Jump PHP</title> 
  26. <style type="text/css"> 
  27. body { background: #FFFFFF; } 
  28. div  { font-family: Verdana, Arial; font-size: 12px; color: #000000; } 
  29. </style> 
  30. </head> 
  31. <body> 
  32. <div> 
  33. Przepraszamy, wystąpiła chwilowa awaria serwisu.<br> 
  34. <br> 
  35. <b>Nie mogę połączyć się z serwerem bazy danych.</b><br> 
  36. <br> 
  37. Proszę powiadom <a href="mailto:qba100@gmail.com">Wemastera qba100@gmail.com</a>. 
  38. </div> 
  39. </html>'); 
  40.  
  41.  
  42. @mysql_select_db($_CONFIG['MySQL']['base']) or die('<html> 
  43. <head> 
  44.  <meta http-equiv="Content-type" content="text/html; charset=iso-8859-2"> 
  45. <title>Error</title> 
  46. <style type="text/css"> 
  47. body { background: #FFFFFF; } 
  48. div  { font-family: Verdana, Arial; font-size: 12px; color: #000000;} 
  49. </style> 
  50. </head> 
  51. <body> 
  52. <div> 
  53. Przepraszamy, wystąpiła chwilowa awaria serwisu.<br> 
  54. <br> 
  55. <b>Nie mogę odnaleść bazy Danych.</b><br> 
  56. <br> 
  57. Proszę powiadom <a href="mailto:qba100@gmail.com">Wemastera qba100@gmail.com</a>. 
  58.  </div> 
  59. </html>'); 
  60.  
  61. function login() { 
  62. global $_CONFIG; 
  63.  
  64.  
  65.  if ('ok' == $_COOKIE['zalogowany'] AND  md5('ok') != $_SESSION['zalogowany']){
  66.  $_SESSION['zalogowany'] = md5('ok') ;
  67.  }
  68.  if (md5('ok') != $_COOKIE['zalogowany'] AND  md5('ok') != $_SESSION['zalogowany']){
  69.  
  70.  
  71. if(isset($_POST['login']) AND isset($_POST['haslo'])) { 
  72. $login = md5(strtolower($_POST['login'])); 
  73. $haslo = md5($_POST['haslo']); 
  74. $metod = "post"; 
  75. }
  76. elseif(isset($_COOKIE['JumpPHP_'.md5('login')]) AND isset($_COOKIE['JumpPHP_'.md5('haslo')])) { 
  77.  $login = $_COOKIE['JumpPHP_'.md5('login')]; 
  78. $haslo = $_COOKIE['JumpPHP_'.md5('haslo')]; 
  79. $metod = "cookies"; 
  80. } 
  81.  
  82. if(isset($metod)) { 
  83. $query = mysql_query("SELECT `user`,`pass`,`nick`,`lvl`,`u1` ,`u2` ,`u3` ,`u4` ,`u5` ,`u6`,`u7` ,
    `nazwisko` ,`email` ,`data`  FROM `users` WHERE `user`='"    .$login."' AND `pass`='".$haslo."';"); 
  84.  if(mysql_num_rows($query) == 1) { 
  85.   $wiersz = mysql_fetch_array($query);
  86.   $_SESSION['zalogowany'] = md5('ok'); 
  87.    $_SESSION['login'] = $wiersz['user']; 
  88.   $_SESSION['haslo'] = $wiersz['pass']; 
  89.   $_SESSION['nick'] = $wiersz['nick']; 
  90.   $_SESSION['u1'] = $wiersz['u1']; 
  91.   $_SESSION['u2'] = $wiersz['u2']; 
  92.   $_SESSION['u3'] = $wiersz['u3']; 
  93.   $_SESSION['u4'] = $wiersz['u4']; 
  94.   $_SESSION['u5'] = $wiersz['u5']; 
  95.   $_SESSION['u6'] = $wiersz['u6']; 
  96.   $_SESSION['u7'] = $wiersz['u7']; 
  97.    $_SESSION['nazwisko'] = $wiersz['nazwisko']; 
  98.   $_SESSION['data'] = $wiersz['data']; 
  99.   $_SESSION['email'] = $wiersz['email']; 
  100.   $_SESSION['lvl'] = $wiersz['lvl']; 
  101.   }
  102.  
  103. 	  } 
  104.  }
  105.  
  106. }
  107. ?>
[PHP] pobierz, plaintext

Plik ten znajduje się w folderze z .htaccess więc nikt nie ma do niego dostępu i inculduję go do innych plików. Co mogę zrobić by poprawić jescze bezpieczeństwo tego skryptu?questionmark.gif


--------------------
Qbix Media Project - Prosty Multiplatformowy Odtwarzacz Muzyki
Go to the top of the page
+Quote Post
ayeo
post
Post #2





Grupa: Przyjaciele php.pl
Postów: 1 202
Pomógł: 117
Dołączył: 13.04.2007
Skąd: 127.0.0.1

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

Po co przechowujesz hasło i login w ciasteczkach? To napewno nie jest bezpieczne. Poza tym przechowywanie zminnych sesyjnych w plikach też nie jest najlepsze jeśli trzymasz tam hasła itd. Naprawdę trzymasz zakodowany login w bazie? smile.gif


--------------------
ayeo.pl
Go to the top of the page
+Quote Post
qba10
post
Post #3





Grupa: Zarejestrowani
Postów: 264
Pomógł: 20
Dołączył: 19.08.2007
Skąd: Bryńsk k. Lidzbarka Welskiego (nie mylić z Warmińskim)

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

no fakt trzymanie hasel i loginu w sesesjach jest zbędne
ale cookie jest potrzebne do zapamietania osoby by się ciągle logować nie musiała


PS: Jak kodować to już na maksa smile.gif


--------------------
Qbix Media Project - Prosty Multiplatformowy Odtwarzacz Muzyki
Go to the top of the page
+Quote Post
ayeo
post
Post #4





Grupa: Przyjaciele php.pl
Postów: 1 202
Pomógł: 117
Dołączył: 13.04.2007
Skąd: 127.0.0.1

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

Standardowo w w COOKIE (PHP robi to automatycznie) trzyma się identyfikator sesji poprostu. Na tej podstawie PHP kojarzy później kolejne wywołania i zmienne smile.gif


--------------------
ayeo.pl
Go to the top of the page
+Quote Post
« Następny starszy · PHP · 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 Aktualny czas: 19.08.2025 - 03:18
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn