mirror of
https://github.com/phpbb/phpbb.git
synced 2025-06-07 20:08:53 +00:00
Merge pull request #5818 from marc1706/ticket/16266
[ticket/16266] Remove extra declaration of constants
This commit is contained in:
commit
d4293866b3
1 changed files with 3 additions and 11 deletions
|
@ -37,23 +37,15 @@ class argon2i extends base_native
|
|||
{
|
||||
parent::__construct($config, $helper);
|
||||
|
||||
// Workaround to prevent "Use of undefined constant" warning on some unsupported PHP installations
|
||||
if (!defined('PASSWORD_ARGON2I'))
|
||||
{
|
||||
define('PASSWORD_ARGON2_DEFAULT_MEMORY_COST', 1024);
|
||||
define('PASSWORD_ARGON2_DEFAULT_TIME_COST', 2);
|
||||
define('PASSWORD_ARGON2_DEFAULT_THREADS', 1);
|
||||
}
|
||||
|
||||
/**
|
||||
* For Sodium implementation of argon2 algorithm (since PHP 7.4), set special value of 1 for "threads" cost factor
|
||||
* See https://wiki.php.net/rfc/sodium.argon.hash and PHPBB3-16266
|
||||
* Don't allow cost factors to be below default settings where possible
|
||||
*/
|
||||
$this->memory_cost = max($memory_cost, PASSWORD_ARGON2_DEFAULT_MEMORY_COST);
|
||||
$this->time_cost = max($time_cost, PASSWORD_ARGON2_DEFAULT_TIME_COST);
|
||||
$this->memory_cost = max($memory_cost, defined('PASSWORD_ARGON2_DEFAULT_MEMORY_COST') ? PASSWORD_ARGON2_DEFAULT_MEMORY_COST : 1024);
|
||||
$this->time_cost = max($time_cost, defined('PASSWORD_ARGON2_DEFAULT_TIME_COST') ? PASSWORD_ARGON2_DEFAULT_TIME_COST : 2);
|
||||
$this->threads = (defined('PASSWORD_ARGON2_PROVIDER') && PASSWORD_ARGON2_PROVIDER == 'sodium') ?
|
||||
PASSWORD_ARGON2_DEFAULT_THREADS : max($threads, PASSWORD_ARGON2_DEFAULT_THREADS);
|
||||
PASSWORD_ARGON2_DEFAULT_THREADS : max($threads, defined('PASSWORD_ARGON2_DEFAULT_THREADS') ? PASSWORD_ARGON2_DEFAULT_THREADS : 1);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
Loading…
Add table
Reference in a new issue