Я пытаюсь хранить пароли пользователей с помощью phpass, но я стараюсь его правильно реализовать. У меня был рабочий код, но я сохранял свой пароль в виде текста, который, как я узнал, небезопасен. Согласно phpbestpractices.org, самым безопасным способом является phpass, поэтому я пошел. Я попробовал прочитать их учебник, но он немного опережает мой диапазон навыков. Изменения, внесенные мной из моего исходного кода, можно увидеть здесь: http://www.diffchecker.com/6gw4g2ps Я заранее извиняюсь, если я полностью зарубил его, я все еще в процессе обучения, но вы можете доверять мне, Я возьму любой совет, которому я могу следовать! Я попытался переместить include и $ hash_obj в инструкцию else, но потом он говорит мне:
Предупреждение: include (/home/carlton/public_html/PHPproject/includes/PasswordHash.php): не удалось открыть поток: разрешение отклонено в /home/carlton/public_html/PHPproject/forms/registersecure.php в строке 40
Предупреждение: include (): Не удалось открыть '/home/carlton/public_html/PHPproject/includes/PasswordHash.php' для включения (include_path = '.: / Usr / share / php: / usr / share / pear') в / home /carlton/public_html/PHPproject/forms/registersecure.php в строке 40
Неустранимая ошибка: Class 'PasswordHash' не найден в /home/carlton/public_html/PHPproject/forms/registersecure.php в строке 41 "
<?php include ('/home/carlton/public_html/PHPproject/includes/PasswordHash.php'); $hash_obj = new PasswordHash(8, false); error_reporting(E_ALL); ini_set('display_errors', 1); if (empty($_POST)){ ?> <form name="registration" action="registersecure.php" method="POST"> <label for "username">Username: </label> <input type="text" name="username"/><br /> <label for "password">Password: </label> <input type="password" name="password"/><br /> <label for "fname">First Name: </label> <input type="text" name="fname"/><br /> <label for "lname">Last name: </label> <input type="text" name="lname"/><br /> <label for "email">Email: </label> <input type="text" name="email"/><br /> <button type="submit">Submit</button> </form> <?php } else{ $form = $_POST; $username = $form['username']; $password = $form['password']; $fname = $form['fname']; $lname = $form['lname']; $email = $form['email']; $user = 'root'; $pass = 'pdt1848!'; if (strlen($password)>72){die("Password must be less than 73 characters."); } $hash = $hash_obj->HashPassword($password); //if (strlen($hash)>=20){ // $sql = "INSERT INTO users (password)VALUE($hash) "; // $query = $db->prepare($sql); // $query->execute(array(':password'=>$hash)); //}; $db = new PDO('mysql:host=localhost;dbname=phpproject', $user, $pass); $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $sql = "INSERT INTO users (username, password, fname, lname, email)VALUES(:username, :password, :fname, :lname, :email)"; $query = $db->prepare($sql); $result = $query->execute(array(':username'=>$username, ':password:'=>$hash, ':fname'=>$fname, ':lname'=>$lname, ':email'=>$email)); if ($result){ echo "Thanks for registering with us!"; } else { echo "Sorry, an error occurred while editing the database. Contact the guy who built this garbage."; }; }; ?> PasswordHash.php
<?php class PasswordHash { var $itoa64; var $iteration_count_log2; var $portable_hashes; var $random_state; function PasswordHash($iteration_count_log2, $portable_hashes) { $this->itoa64 = './0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz'; if ($iteration_count_log2 < 4 || $iteration_count_log2 > 31) $iteration_count_log2 = 8; $this->iteration_count_log2 = $iteration_count_log2; $this->portable_hashes = $portable_hashes; $this->random_state = microtime(); if (function_exists('getmypid')) $this->random_state .= getmypid(); } function get_random_bytes($count) { $output = ''; if (is_readable('/dev/urandom') && ($fh = @fopen('/dev/urandom', 'rb'))) { $output = fread($fh, $count); fclose($fh); } if (strlen($output) < $count) { $output = ''; for ($i = 0; $i < $count; $i += 16) { $this->random_state = md5(microtime() . $this->random_state); $output .= pack('H*', md5($this->random_state)); } $output = substr($output, 0, $count); } return $output; } function encode64($input, $count) { $output = ''; $i = 0; do { $value = ord($input[$i++]); $output .= $this->itoa64[$value & 0x3f]; if ($i < $count) $value |= ord($input[$i]) << 8; $output .= $this->itoa64[($value >> 6) & 0x3f]; if ($i++ >= $count) break; if ($i < $count) $value |= ord($input[$i]) << 16; $output .= $this->itoa64[($value >> 12) & 0x3f]; if ($i++ >= $count) break; $output .= $this->itoa64[($value >> 18) & 0x3f]; } while ($i < $count); return $output; } function gensalt_private($input) { $output = '$P$'; $output .= $this->itoa64[min($this->iteration_count_log2 + ((PHP_VERSION >= '5') ? 5 : 3), 30)]; $output .= $this->encode64($input, 6); return $output; } function crypt_private($password, $setting) { $output = '*0'; if (substr($setting, 0, 2) == $output) $output = '*1'; $id = substr($setting, 0, 3); # We use "$P$", phpBB3 uses "$H$" for the same thing if ($id != '$P$' && $id != '$H$') return $output; $count_log2 = strpos($this->itoa64, $setting[3]); if ($count_log2 < 7 || $count_log2 > 30) return $output; $count = 1 << $count_log2; $salt = substr($setting, 4, 8); if (strlen($salt) != 8) return $output; # We're kind of forced to use MD5 here since it's the only # cryptographic primitive available in all versions of PHP # currently in use. To implement our own low-level crypto # in PHP would result in much worse performance and # consequently in lower iteration counts and hashes that are # quicker to crack (by non-PHP code). if (PHP_VERSION >= '5') { $hash = md5($salt . $password, TRUE); do { $hash = md5($hash . $password, TRUE); } while (--$count); } else { $hash = pack('H*', md5($salt . $password)); do { $hash = pack('H*', md5($hash . $password)); } while (--$count); } $output = substr($setting, 0, 12); $output .= $this->encode64($hash, 16); return $output; } function gensalt_extended($input) { $count_log2 = min($this->iteration_count_log2 + 8, 24); # This should be odd to not reveal weak DES keys, and the # maximum valid value is (2**24 - 1) which is odd anyway. $count = (1 << $count_log2) - 1; $output = '_'; $output .= $this->itoa64[$count & 0x3f]; $output .= $this->itoa64[($count >> 6) & 0x3f]; $output .= $this->itoa64[($count >> 12) & 0x3f]; $output .= $this->itoa64[($count >> 18) & 0x3f]; $output .= $this->encode64($input, 3); return $output; } function gensalt_blowfish($input) { # This one needs to use a different order of characters and a # different encoding scheme from the one in encode64() above. # We care because the last character in our encoded string will # only represent 2 bits. While two known implementations of # bcrypt will happily accept and correct a salt string which # has the 4 unused bits set to non-zero, we do not want to take # chances and we also do not want to waste an additional byte # of entropy. $itoa64 = './ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789'; $output = '$2a$'; $output .= chr(ord('0') + $this->iteration_count_log2 / 10); $output .= chr(ord('0') + $this->iteration_count_log2 % 10); $output .= '$'; $i = 0; do { $c1 = ord($input[$i++]); $output .= $itoa64[$c1 >> 2]; $c1 = ($c1 & 0x03) << 4; if ($i >= 16) { $output .= $itoa64[$c1]; break; } $c2 = ord($input[$i++]); $c1 |= $c2 >> 4; $output .= $itoa64[$c1]; $c1 = ($c2 & 0x0f) << 2; $c2 = ord($input[$i++]); $c1 |= $c2 >> 6; $output .= $itoa64[$c1]; $output .= $itoa64[$c2 & 0x3f]; } while (1); return $output; } function HashPassword($password) { $random = ''; if (CRYPT_BLOWFISH == 1 && !$this->portable_hashes) { $random = $this->get_random_bytes(16); $hash = crypt($password, $this->gensalt_blowfish($random)); if (strlen($hash) == 60) return $hash; } if (CRYPT_EXT_DES == 1 && !$this->portable_hashes) { if (strlen($random) < 3) $random = $this->get_random_bytes(3); $hash = crypt($password, $this->gensalt_extended($random)); if (strlen($hash) == 20) return $hash; } if (strlen($random) < 6) $random = $this->get_random_bytes(6); $hash = $this->crypt_private($password, $this->gensalt_private($random)); if (strlen($hash) == 34) return $hash; # Returning '*' on error is safe here, but would _not_ be safe # in a crypt(3)-like function used _both_ for generating new # hashes and for validating passwords against existing hashes. return '*'; } function CheckPassword($password, $stored_hash) { $hash = $this->crypt_private($password, $stored_hash); if ($hash[0] == '*') $hash = crypt($password, $stored_hash); return $hash == $stored_hash; } } ?>