Controlabs\Helper\Password

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	encrypt()	0	10	2
A	verify()	0	4	1

<?php

namespace Controlabs\Helper;

class Password
{
    protected const SHA_512 = 'sha512';

    public function encrypt(string $password, string $salt = null, string $algorithm = self::SHA_512): PasswordData
    {
        if (!$salt) {

The expression $salt of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $salt = hash($algorithm, uniqid(strval(mt_rand(1, mt_getrandmax())), true));
        }

        $password = hash($algorithm, $password . $salt);

        return new PasswordData($password, $salt);
    }

    public function verify(string $encrypted, string $password, string $salt, string $algorithm = self::SHA_512): bool
    {
        return $encrypted === $this->encrypt($password, $salt, $algorithm)->password();
    }
}