Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][MYSQL]md5 do systemu rejestracji i logowania
Forum PHP.pl > Forum > Przedszkole
Tomaszu
22.02.2009, 21:03:37
Witam,
Na wstępie poinformuję, że jestem tutaj nowy i niestety nie jestem zaawansowanym w programowaniu php. Nie jestem natomiast też całkiem zielony smile.gif

Do rzeczy:
Mam skrypt systemu rejestracji, w którym chciałbym aby zapisywane hasło w db(MySQL) było hash'owane przez md5 a później podczas logowania hash wpisanego hasła porównywany był z tym wcześniej zapisanym w db. - chyba wiadomo o co chodzi?

Wygląda to tak:

join.php - dopisujący do bazy użytkowników:
Kod
<?php
  error_reporting(E_ALL);
  include_once("func.php");
  session_register("login");
  session_register("password");
  session_register("loggedIn");
  $messages=array();
  $dbhost="localhost";
  $dbuser="xxx";
  $dbpass="xxx";
  $dbname="xxx";
  connectToDB();
  checkLoggedIn("no");
  if(isset($_POST["submit"])){
      field_validator("Login", $_POST["login"], "alphanumeric", 4, 15);
      field_validator("Hasło", $_POST["password"], "string", 4, 15);
      field_validator("Powtórzenie hasła", $_POST["password2"], "string", 4, 15);
      if(strcmp($_POST["password"], $_POST["password2"])) {
          $messages[]="Powtórzone hasła nie pasują";
      }
      $query="SELECT login FROM users WHERE login='".$_POST["login"]."'";
      $result=mysql_query($query, $link) or die("MySQL query $query failed.  Error if any: ".mysql_error());
      if( ($row=mysql_fetch_array($result)) ){
          $messages[]="Login ID \"".$_POST["login"]."\" already exists.  Try another.";
      }
      if(empty($messages)) {
          newUser($_POST["login"], $_POST["password"]);
          cleanMemberSession($_POST["login"], $_POST["password"]);
          header("Location: members.php?".session_name()."=".session_id());
      }
  }
  ?>
  <html>
  <head>
  <META HTTP-EQUIV="Content-type" CONTENT="text/html; charset=iso-8859-2">
  <META HTTP-EQUIV="Reply-to" CONTENT="t.rosciszewski@gmail.com">
  <META HTTP-EQUIV="Content-Language" CONTENT="pl">
  <META NAME="Author" CONTENT="Tomasz Rościszewski">
  <style type="text/css">
  body{font-family: Arial, Helvetica; font-size: 10pt}
  h1{font-size: 12pt}
  </style>
  </head>
  <body>
  <?php
  if(!empty($messages)){
      displayErrors($messages);
  }
  ?>
  <form action="<?=$_SERVER["PHP_SELF"]?>" method="POST">
  <TABLE BORDER="0">
  <TR>
      <TD>Login:
      </td>
      <td>
          <input type="text" name="login" value="<?php print isset($_POST["login"]) ? $_POST["login"] : "";?>" maxlength="15">
      </td>
  </tr>
  <tr>
      <td>Password:
      </td>
      <td>
          <input type="password" name="password" value="" maxlength="15">
      </td>
  </tr>
  <tr>
      <td>Confirm password:
      </td>
      <td>
          <input type="password" name="password2" value="" maxlength="15">
      </td>
  </tr>
  <tr>
      <td>&nbsp;
      </td>
      <td>
          <input name="submit" type="submit" value="Submit">
      </td>
  </tr>
  </table>
  </form>
  </body>
  </html>


login.php - logujący:
Kod
<?php
error_reporting(E_ALL);
include_once("func.php");
session_register("login");
session_register("password");
session_register("loggedIn");
$messages=array();
$dbhost="localhost";
$dbuser="xxx";
$dbpass="xxx";
$dbname="xxx";
connectToDB();
checkLoggedIn("no");
if(isset($_POST["submit"])) {
     field_validator("Login", $_POST["login"], "alphanumeric", 4, 15);
     field_validator("Hasło", $_POST["password"], "string", 4, 15);
     if($messages){
         doIndex();
         exit;
     }
     if( !($row = checkPass($_POST["login"], $_POST["password"])) ) {
         $messages[]="Incorrect login/password, try again";
     }
     if($messages){
         doIndex();
         exit;
     }
     cleanMemberSession($row["login"], $row["password"]);
     header("Location: members.php?".session_name()."=".session_id());
} else {  
     doIndex();
}
function doIndex() {
     global $messages;
     global $title;
?>
<html>
<head>
<META HTTP-EQUIV="Content-type" CONTENT="text/html; charset=iso-8859-2">
<META HTTP-EQUIV="Content-Language" CONTENT="pl">
<style type="text/css">
body{font-family: Arial, Helvetica; font-size: 10pt}
h1{font-size: 12pt}
</style>
</head>
<body>
<h1><?=$title?></h1>
<?php
if($messages) { displayErrors($messages); }
?>
<form action="<?=$_SERVER["PHP_SELF"]?>" method="POST">
<table>
<tr>
<td>Login:</td>
<td><input type="text" name="login" value="<?php print isset($_POST["login"]) ? $_POST["login"] : "";?>" maxlength="15"></td>
</tr>
<tr>
<td>Password:</td>
<td><input type="password" name="password" value="" maxlength="15"></td>
</tr>
<tr>
<td>&nbsp;</td>
<td><input name="submit" type="submit" value="Submit"></td>
</tr>
</table>
</form>
</body>
</html>
<?php
}
?>


oraz func.php includowany w poprzednich:
Kod
<?php
function connectToDB() {
     global $link, $dbhost, $dbuser, $dbpass, $dbname;  
     ($link = mysql_pconnect("$dbhost", "$dbuser", "$dbpass")) || die("Couldn't connect to MySQL");
     mysql_select_db("$dbname", $link) || die("Nie mogę otworzyć bazy danych. Błędy: ".mysql_error() );
}
function newUser($login, $password) {
     global $link;
     $query="INSERT INTO users (login, password) VALUES('$login', '$password')";
     $result=mysql_query($query, $link) or die("Błędy podczas zapisywania danych w bazie: ".mysql_error());

     return true;
}
function displayErrors($messages) {
     print("<b><font color=orange>Wystąpiły następujące problemy:</FONT></b>\n<ul>\n");
     foreach($messages as $msg){
         print("<li><font color=orange>$msg</Font></li>\n");
     }
     print("</ul>\n");
}
function checkLoggedIn($status){
     switch($status){
         case "yes":
             if(!isset($_SESSION["loggedIn"])){
                 header("Location: login.php");
                 exit;
             }
             break;
         case "no":
             if(isset($_SESSION["loggedIn"]) && $_SESSION["loggedIn"] === true ){
                 header("Location: members.php?".session_name()."=".session_id());
             }
             break;          
     }  
     return true;
}
function checkPass($login, $password) {
     global $link;
     $query="SELECT login, password FROM users WHERE login='$login' and password='$password'";
     $result=mysql_query($query, $link)
         or die("checkPass fatal error: ".mysql_error());
     if(mysql_num_rows($result)==1) {
         $row=mysql_fetch_array($result);
         return $row;
     }
     return false;
}
function cleanMemberSession($login, $password) {
     $_SESSION["login"]=$login;
     $_SESSION["password"]=$password;
     $_SESSION["loggedIn"]=true;
}
function flushMemberSession() {
     unset($_SESSION["login"]);
     unset($_SESSION["password"]);
     unset($_SESSION["loggedIn"]);
     session_destroy();
     return true;
}
function field_validator($field_descr, $field_data,
   $field_type, $min_length="", $max_length="",
   $field_required=1) {
     global $messages;
     if(!$field_data && !$field_required){ return; }
     $field_ok=false;
     $email_regexp="^([a-zA-Z0-9_\-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|";
     $email_regexp.="(([a-zA-Z0-9\-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\]?)$";
     $data_types=array(
         "email"=>$email_regexp,
         "digit"=>"^[0-9]$",
         "number"=>"^[0-9]+$",
         "alpha"=>"^[a-zA-Z]+$",
         "alpha_space"=>"^[a-zA-Z ]+$",
         "alphanumeric"=>"^[a-zA-Z0-9]+$",
         "alphanumeric_space"=>"^[a-zA-Z0-9 ]+$",
         "string"=>""
     );
     if ($field_required && empty($field_data)) {
         $messages[] = "$field_descr jest polem wymaganym.";
         return;
     }
     if ($field_type == "string") {
         $field_ok = true;
     } else {
         $field_ok = ereg($data_types[$field_type], $field_data);
     }
     if (!$field_ok) {
         $messages[] = "Wpisz poprawnie pole $field_descr.";
         return;
     }
     if ($field_ok && ($min_length > 0)) {
         if (strlen($field_data) < $min_length) {
             $messages[] = "$field_descr jest nieprawidłowe, musi posiadać minimum $min_length litery.";
             return;
         }
     }
     if ($field_ok && ($max_length > 0)) {
         if (strlen($field_data) > $max_length) {
             $messages[] = "$field_descr jest nieprawidłowe, musi posiadać nie więcej niż $max_length liter.";
             return;
         }
     }
}
?>



Ma ktoś pomysł?
kefirek
22.02.2009, 21:04:57
W pliku func.php zamien na to
[PHP] pobierz, plaintext 
  1. <?php
  2. query="SELECT login, password FROM users WHERE login='$login' and password='".md5($password)."'";
  3. ?>
[PHP] pobierz, plaintext


oraz

[PHP] pobierz, plaintext 
  1. <?php
  2. $query="INSERT INTO users (login, password) VALUES('$login', '".md5($password)."')";
  3. ?>
[PHP] pobierz, plaintext
Foxx
22.02.2009, 21:09:21
Dodam jeszcze, że prawidłowe zagnieżdżanie zmiennych w zapytaniu sql wygląda tak:
[PHP] pobierz, plaintext 
  1. <?php
  2. $query="SELECT login, password FROM users WHERE login='".$login."' and password='".md5($password)."'";
  3. ?>
[PHP] pobierz, plaintext
DREEMus
22.02.2009, 21:09:28
A na przyszłość nie zapomni o poprawnym BBCode ;D
Tomaszu
22.02.2009, 21:25:45
Cytat(DREEMus @ 22.02.2009, 21:09:28 ) *
A na przyszłość nie zapomni o poprawnym BBCode ;D


będę się starał. smile.gif


dodałem to co polecilił kefirek i :
- zapisuje w bazie danych hasło przez md5
- przy logowaniu nie porównuje prawidłowo hasła. Nie da się zalogować.


:/
guilty82
22.02.2009, 21:30:48 
[PHP] pobierz, plaintext 
  1. <?php
  2. ...
  3. if( !($row = checkPass($_POST["login"], md5($_POST["password"]))) ) {
  4. ...
  5. ?>
[PHP] pobierz, plaintext
Tomaszu
22.02.2009, 21:39:31
niestety nie pomogło...
Foxx
22.02.2009, 21:49:09
Cytat
przy logowaniu nie porównuje prawidłowo hasła. Nie da się zalogować.

Spróbuj sam zidentyfikować błąd.
Sprawdź których ifów warunki powinny być spełnione, na przykład ten:
[PHP] pobierz, plaintext 
  1. <?php
  2. if( !($row = checkPass($_POST["login"], md5($_POST["password"]))) ) {
  3. ?>
[PHP] pobierz, plaintext

Nie powinien być a prawdopodobnie jest. Jeżeli tak to sprawdź dlaczego funkcja checkPass nie zwraca rekordu.
Czy jest błąd w zapytaniu SQL, sprawdź surowe dane w bazie - czy są takie jak oczekujesz.
A może nie chodzi o to miejsce, zidentyfikuj je.
Tomaszu
22.02.2009, 22:12:07
Cytat(Foxx @ 22.02.2009, 21:49:09 ) *
Nie powinien być a prawdopodobnie jest. Jeżeli tak to sprawdź dlaczego funkcja checkPass nie zwraca rekordu.
Czy jest błąd w zapytaniu SQL, sprawdź surowe dane w bazie - czy są takie jak oczekujesz.
A może nie chodzi o to miejsce, zidentyfikuj je.


no nie mam pomysłu ale wydaje mi się że szwankuje sama funkcja checkPass... 

[PHP] pobierz, plaintext 
  1. <?php
  2. [size="2"]function checkPass($login, $password) {
  3.  
  4. global $link;
  5.  
  6. $query="SELECT login, password FROM users WHERE login='$login' and password='".md5($password)."'";
  7.  
  8. $result=mysql_query($query, $link)
  9.  
  10. or die("checkPass fatal error: ".mysql_error());
  11.  
  12. if(mysql_num_rows($result)==1) {
  13.  
  14. $row=mysql_fetch_array($result);
  15.  
  16. return $row;
  17.  
  18. }
  19.  
  20. return false;
  21.  
  22. }
  23.  
  24. [/size]
  25. ?>
[PHP] pobierz, plaintext
Foxx
22.02.2009, 22:33:53
Ok. Więc postaw echo przed $query, obejrzyj jaką formę ostatecznie przyjmuje zapytanie.
Czy zwraca w ogóle jakieś rekordy, czy występuje jakiś błąd składni sql.
Czy w bazie jest rekord, który powinien zostać zwrócony.
Wklej do phpmyadmina zapytanie i sprawdź czy w ten sposób otrzymujesz jakieś wyniki z bazy.
Diagnozuj, diagnozuj smile.gif
Tomaszu
23.02.2009, 21:02:03
Posiedziałem nad tym troszkę i powód nie działanie się odnalazł...
Porównałem md5 u mnie i w zewnętrznym generatorze. Sprawa była prosta: 
[SQL] pobierz, plaintext 
  1. password varchar(15) DEFAULT '0' ,
[SQL] pobierz, plaintext
smile.gif

Ucinało mi md5 po 15tym znaku.
Tak utworzyłem tabelę i zapomniałem o tym.
Za moje utrudnianie przepraszam i dziękuję Panowie.



Acha, i dla tych którzy chcieliby tego użyć:

Nie potrzebne też jest w pliku login.php:
[PHP] pobierz, plaintext 
  1. <?php
  2. if( !($row = checkPass($_POST["login"], md5($_POST["password"]))) ) {
  3. ?>
[PHP] pobierz, plaintext


natomiast samo:
[PHP] pobierz, plaintext 
  1. <?php
  2. if( !($row = checkPass($_POST["login"], $_POST["password"])) ) {
  3. ?>
[PHP] pobierz, plaintext


password jest hashowane już w samej funkcji checkPass.
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.