@@ -11,39 +11,39 @@ |
||
| 11 | 11 | */ |
| 12 | 12 | class PBKDF2 extends PasswordEncryptor_PHPHash |
| 13 | 13 | { |
| 14 | - /** |
|
| 15 | - * The number of internal iterations for hash_pbkdf2() to perform for the derivation. Please note that if you |
|
| 16 | - * change this from the default value you will break existing hashes stored in the database, so these would |
|
| 17 | - * need to be regenerated. |
|
| 18 | - * |
|
| 19 | - * @var int |
|
| 20 | - */ |
|
| 21 | - protected $iterations = 10000; |
|
| 14 | + /** |
|
| 15 | + * The number of internal iterations for hash_pbkdf2() to perform for the derivation. Please note that if you |
|
| 16 | + * change this from the default value you will break existing hashes stored in the database, so these would |
|
| 17 | + * need to be regenerated. |
|
| 18 | + * |
|
| 19 | + * @var int |
|
| 20 | + */ |
|
| 21 | + protected $iterations = 10000; |
|
| 22 | 22 | |
| 23 | - /** |
|
| 24 | - * @param string $algorithm |
|
| 25 | - * @param int|null $iterations |
|
| 26 | - * @throws Exception If the provided algorithm is not available in the current environment |
|
| 27 | - */ |
|
| 28 | - public function __construct(string $algorithm, int $iterations = null) |
|
| 29 | - { |
|
| 30 | - parent::__construct($algorithm); |
|
| 23 | + /** |
|
| 24 | + * @param string $algorithm |
|
| 25 | + * @param int|null $iterations |
|
| 26 | + * @throws Exception If the provided algorithm is not available in the current environment |
|
| 27 | + */ |
|
| 28 | + public function __construct(string $algorithm, int $iterations = null) |
|
| 29 | + { |
|
| 30 | + parent::__construct($algorithm); |
|
| 31 | 31 | |
| 32 | - if ($iterations !== null) { |
|
| 33 | - $this->iterations = $iterations; |
|
| 34 | - } |
|
| 35 | - } |
|
| 32 | + if ($iterations !== null) { |
|
| 33 | + $this->iterations = $iterations; |
|
| 34 | + } |
|
| 35 | + } |
|
| 36 | 36 | |
| 37 | - /** |
|
| 38 | - * @return int |
|
| 39 | - */ |
|
| 40 | - public function getIterations(): int |
|
| 41 | - { |
|
| 42 | - return $this->iterations; |
|
| 43 | - } |
|
| 37 | + /** |
|
| 38 | + * @return int |
|
| 39 | + */ |
|
| 40 | + public function getIterations(): int |
|
| 41 | + { |
|
| 42 | + return $this->iterations; |
|
| 43 | + } |
|
| 44 | 44 | |
| 45 | - public function encrypt($password, $salt = null, $member = null) |
|
| 46 | - { |
|
| 47 | - return hash_pbkdf2($this->getAlgorithm(), (string) $password, (string) $salt, $this->getIterations()); |
|
| 48 | - } |
|
| 45 | + public function encrypt($password, $salt = null, $member = null) |
|
| 46 | + { |
|
| 47 | + return hash_pbkdf2($this->getAlgorithm(), (string) $password, (string) $salt, $this->getIterations()); |
|
| 48 | + } |
|
| 49 | 49 | } |
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | { |
| 30 | 30 | parent::__construct($algorithm); |
| 31 | 31 | |
| 32 | - if ($iterations !== null) { |
|
| 32 | + if ($iterations!==null) { |
|
| 33 | 33 | $this->iterations = $iterations; |
| 34 | 34 | } |
| 35 | 35 | } |
@@ -44,6 +44,6 @@ discard block |
||
| 44 | 44 | |
| 45 | 45 | public function encrypt($password, $salt = null, $member = null) |
| 46 | 46 | { |
| 47 | - return hash_pbkdf2($this->getAlgorithm(), (string) $password, (string) $salt, $this->getIterations()); |
|
| 47 | + return hash_pbkdf2($this->getAlgorithm(), (string)$password, (string)$salt, $this->getIterations()); |
|
| 48 | 48 | } |
| 49 | 49 | } |
@@ -8,30 +8,30 @@ |
||
| 8 | 8 | |
| 9 | 9 | class PBKDF2Test extends SapphireTest |
| 10 | 10 | { |
| 11 | - public function testGetIterations() |
|
| 12 | - { |
|
| 13 | - $encryptor = new PBKDF2('sha512', 12345); |
|
| 14 | - $this->assertSame(12345, $encryptor->getIterations()); |
|
| 15 | - } |
|
| 11 | + public function testGetIterations() |
|
| 12 | + { |
|
| 13 | + $encryptor = new PBKDF2('sha512', 12345); |
|
| 14 | + $this->assertSame(12345, $encryptor->getIterations()); |
|
| 15 | + } |
|
| 16 | 16 | |
| 17 | - public function testEncrypt() |
|
| 18 | - { |
|
| 19 | - $encryptor = new PBKDF2('sha512', 10000); |
|
| 20 | - $salt = 'predictablesaltforunittesting'; |
|
| 21 | - $result = $encryptor->encrypt('opensesame', $salt); |
|
| 22 | - $this->assertSame( |
|
| 23 | - '6bafcacb90', |
|
| 24 | - substr($result, 0, 10), |
|
| 25 | - 'Hashed password with predictable salt did not match fixtured expectation' |
|
| 26 | - ); |
|
| 27 | - } |
|
| 17 | + public function testEncrypt() |
|
| 18 | + { |
|
| 19 | + $encryptor = new PBKDF2('sha512', 10000); |
|
| 20 | + $salt = 'predictablesaltforunittesting'; |
|
| 21 | + $result = $encryptor->encrypt('opensesame', $salt); |
|
| 22 | + $this->assertSame( |
|
| 23 | + '6bafcacb90', |
|
| 24 | + substr($result, 0, 10), |
|
| 25 | + 'Hashed password with predictable salt did not match fixtured expectation' |
|
| 26 | + ); |
|
| 27 | + } |
|
| 28 | 28 | |
| 29 | - /** |
|
| 30 | - * @expectedException Exception |
|
| 31 | - * @expectedExceptionMessage Hash algorithm "foobar" not found |
|
| 32 | - */ |
|
| 33 | - public function testThrowsExceptionWhenInvalidAlgorithmIsProvided() |
|
| 34 | - { |
|
| 35 | - new PBKDF2('foobar'); |
|
| 36 | - } |
|
| 29 | + /** |
|
| 30 | + * @expectedException Exception |
|
| 31 | + * @expectedExceptionMessage Hash algorithm "foobar" not found |
|
| 32 | + */ |
|
| 33 | + public function testThrowsExceptionWhenInvalidAlgorithmIsProvided() |
|
| 34 | + { |
|
| 35 | + new PBKDF2('foobar'); |
|
| 36 | + } |
|
| 37 | 37 | } |