@@ -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 | } |