deon/phpldapadmin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix for ARGON2 passwords, they shouldnt be base64 encoded. Fixes #316
All checks were successful
Create Docker Image / Test Application (x86_64) (push) Successful in 27s
Details
Create Docker Image / Build Docker Image (x86_64) (push) Successful in 1m19s
Details
Create Docker Image / Build Docker Image (arm64) (push) Successful in 4m32s
Details
Create Docker Image / Final Docker Image Manifest (push) Successful in 8s
Details

Browse Source
This commit is contained in:
Deon George 2025-05-03 23:37:32 +10:00
parent 2341c54da4
commit 6f63726606
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/Classes/LDAP/Attribute/Password/Argon2i.php
View File

@ -10,16 +10,16 @@ final class Argon2i extends Base
public static function subid(string $password): bool
{
return str_starts_with(base64_decode(self::password($password)),self::identifier.'$');
return str_starts_with(self::password($password),self::identifier.'$');
}
public function compare(string $source,string $compare): bool
{
return password_verify($compare,base64_decode($this->password($source)));
return password_verify($compare,$this->password($source));
}
public function encode(string $password): string
{
return sprintf('{%s}%s',self::key,base64_encode(password_hash($password,PASSWORD_ARGON2I)));
return sprintf('{%s}%s',self::key,password_hash($password,PASSWORD_ARGON2I));
}
}