[PHP]MySQL]Skrypt logowania, Proszę o wyrażenie swojego zdania na ten temat . Opcje

[kamillo121]

Witam , napisałem sobie skrypt logowania i nie wiem czemu ale wydaje mi się, że jest strasznie cienko zabezpieczony , tzn jest to mój drugi skrypt (po próbach ) i nie spodziewam się skryptu nie do obejścia ale takiego chociaż dość dobrego , możecie zerknąć na to i dać swoją opinie ? (tylko bądźcie wyrozumiali dla mnie, proszę (IMG:style_emoticons/default/smile.gif) (wiem, że jest do tego dział ale ja bym w razie czego chciał jakąś pomoc co do tego skryptu, tzn jakieś inne sposoby czy coś ) )

[PHP] pobierz, plaintext 
 
<?php
if(!isset($_REQUEST['action'])){
header('Location: index.php?page=news');
}else{
switch($_REQUEST['action']){
   case 'Zaloguj':
     if(!isset($_POST['login']) OR !isset($_POST['haslo']) OR strlen($_POST['login'])<1 OR strlen($_POST['haslo'])<1){
	 header('Location: index.php?page=news');
	 }else{
	  if(preg_match('/^[a-zA-Z0-9\_\-]{3,20}$/D', $_POST['login']) AND preg_match('/^[a-zA-Z0-9]{4,20}$/D', $_POST['haslo'])){
	  include 'conn.php';
	  $login=stripslashes($_POST['login']);
	  $haslo=stripslashes($_POST['haslo']); 
/// Tutaj dodałem według wskazówek szanownych forumowiczów przepuszczanie przez sha1  hasła :) 
	  $sql="SELECT * FROM user WHERE login='".$login."' AND haslo='".$haslo."' LIMIT 1 ";
	   $result=mysql_query($sql)
	      or die(mysql_error());
		   unset($sql);
	      if($row=mysql_fetch_array($result)){
	 session_start();
	 $_SESSION['login']=$row['login'];
	 $_SESSION['lvl_acc']=$row['lvl_acc'];
	 	$sql="UPDATE user  SET ostatnia_wiz='".date("Y-m-d H:i:s", time())."' " .
										"WHERE id=".$row['id']." ";
										mysql_query($sql)
										  or die(mysql_error());
										  echo '<div id="kom"><div id="tab">Komunikat</div>';
										  echo 'zalogowano<br><br>';
										  echo '<a href="index.php?page=news">Powrót na stronę głowną </a>';
	                                      echo '</div>';
	 }else{
	  echo '<div id="kom"><div id="tab">Komunikat</div>';
	  echo 'Złe hasło bądź login<br> ';
	  echo '<a href="index.php?page=news">Powrót na stronę głowną </a>';
	  echo '</div>';
	  }
	  }else{
	  echo '<div id="kom"><div id="tab">Komunikat</div>';
	  echo 'Złe dane<br> ';
	  echo '<a href="index.php?page=news">Powrót na stronę głowną </a>';
	  echo '</div>';
	  }
	 }
	break;
	case 'Wyloguj':
	session_start();
	session_unset();
	session_destroy();
	echo '<div id="kom"><div id="tab">Komunikat</div>';
	echo 'Wylogowano<br>';
	echo '<a href="index.php?page=news">Powrót na stronę głowną </a>';
	echo '</div>';
	break;
       default:
        header('Location: index.php?page=news');
        break;
 
}
//koniec switch'a
}
?>
[PHP] pobierz, plaintext 

(To jest sam skrypt który otrzymuje dane z formularza, przetwarza i loguje )

Ten post edytował kamillo121 24.08.2009, 15:30:14

[kamillo121]

EEE, że znowu zmienię temat rozmowy ..
Przeczytałem ten artykuł tylko mam takie pytanie ..
Jak mam w tym miejscy wpisać dane dotyczące połączenia

[PHP] pobierz, plaintext 
 
 
//kawałek kodu
  * Konstruktor
  * Ustalanie parametrów i przygotowywanie środowiska pod funkcjonowanie klasy
  * @param string $db_config Dane konfiguracyjne bazy
  * @param string $session_table Tabela z danymi sesyjnymi 
  * @param string $session_lifetime Maksymalny czas istnienia nieaktywnej sesji @default 0 
  * @return void 
  * @access public 
  */ 
  function session($db_config =" '', $session_table, $session_lifetime = 0) 
  {
  if(!empty($db_config)){
  // Ustalanie danych konfiguracyjnych bazy
  $this->db_config['host'] = $db_config['host'];
  $this->db_config['user'] = $db_config['user'];
  $this->db_config['password'] = $db_config['password'];
  $this->db_config['datebase'] = $db_config['datebase'];
  $this->db_config['persistent'] = $db_config['persistent'];
  } 
 
 
 
[PHP] pobierz, plaintext 

Tam w db_config
Tam gdzie jest zmienna $session_table to mam dopisać żeby tak to wyglądało $session_table="nazwa_tabeli" ?>
I potem tak patrze, tam, niżej ktoś napisał, że autor powstawiał bugi , to prawda ?

Ten post edytował kamillo121 24.08.2009, 20:31:46