From 422e97d3299eadc866ae4ae38b7efa86d1e702ba Mon Sep 17 00:00:00 2001 From: rudy Date: Fri, 16 Dec 2022 16:49:24 +0100 Subject: [PATCH] =?UTF-8?q?supression=20des=20fonctions=20li=C3=A9es=20?= =?UTF-8?q?=C3=A0=20l'update=20du=20hashage=20de=20mot=20de=20passe?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .env | 1 - config/services.yaml | 4 - .../compose/postgres/init-db.d/fill_lasql.sql | 2 +- docker-compose.yml | 1 - readme.md | 15 ++-- src/SQLLogin/SQLLoginRequest.php | 11 --- src/Security/Hasher/PasswordEncoder.php | 84 +++---------------- src/Security/SQLLoginUserAuthenticator.php | 8 +- src/Service/SQLLoginService.php | 17 ---- 9 files changed, 26 insertions(+), 117 deletions(-) diff --git a/.env b/.env index 868b9f6..11c6cdf 100644 --- a/.env +++ b/.env @@ -30,7 +30,6 @@ BASE_URL='http://localhost:8080' HYDRA_ADMIN_BASE_URL='http://hydra:4445' APP_LOCALES="fr,en" SECURITY_PATTERN= -NEW_HASH_ALGO= HASH_ALGO_LEGACY="sha256" ###> symfony/lock ### # Choose one of the stores below diff --git a/config/services.yaml b/config/services.yaml index e331ae1..cebc5f0 100644 --- a/config/services.yaml +++ b/config/services.yaml @@ -13,9 +13,6 @@ parameters: env(HASH_ALGO_LEGACY): "sha256" hashAlgoLegacy: '%env(resolve:HASH_ALGO_LEGACY)%' - env(NEW_HASH_ALGO): "sha256" - newHashAlgo: '%env(resolve:NEW_HASH_ALGO)%' - # adresse du site hote issuer_url: '%env(resolve:ISSUER_URL)%' # adresse de hydra @@ -71,7 +68,6 @@ services: arguments: $pepper: '%pepper%' $hashAlgoLegacy: '%hashAlgoLegacy%' - $newHashAlgo: '%newHashAlgo%' $securityPattern: '%security_pattern%' # add more service definitions when explicit configuration is needed # please note that last definitions always *replace* previous ones diff --git a/containers/compose/postgres/init-db.d/fill_lasql.sql b/containers/compose/postgres/init-db.d/fill_lasql.sql index 0284076..f2d0df4 100644 --- a/containers/compose/postgres/init-db.d/fill_lasql.sql +++ b/containers/compose/postgres/init-db.d/fill_lasql.sql @@ -10,6 +10,6 @@ INSERT INTO usager (email, password, salt, lastname, firstname) VALUES ('test3@test.com', '504ae1c3e2f5fdaf41f868164dabcef21e17059f5f388b452718a1ce92692c67', 'cesaltestunautreexemple', 'Dupont', 'Henri'); INSERT INTO usager (email, password, lastname, firstname) VALUES - ('test4@test.com', '$2a$12$zFN0VJ..Cuu.2itWQwmHJe5EUhNHazbMfCSJFpNiEfdwpLzjjDM0u', 'Durand', 'Isabelle'), + ('test4@test.com', '$2a$12$91AHN7WFXieeadvUfZ88mO.9N7oS5adeXbdERnRno9oLAbqqDW4IG', 'Durand', 'Isabelle'), ('test2@test.com', '50626fa21f45a275cea0efff13ff78fd02234cade322da08b7191c7e9150141d', 'Dubois', 'Angela'); GRANT ALL PRIVILEGES ON DATABASE usager TO lasql diff --git a/docker-compose.yml b/docker-compose.yml index 8d13564..ad59579 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -38,7 +38,6 @@ services: - DEFAULT_LOCALE=fr - DSN_REMOTE_DATABASE=pgsql:host='postgres';port=5432;dbname=lasql; - HASH_ALGO_LEGACY=sha256 - - NEW_HASH_ALGO=bcrypt - SECURITY_PATTERN=password,salt,pepper diff --git a/readme.md b/readme.md index 5b04de2..361fbc2 100644 --- a/readme.md +++ b/readme.md @@ -34,23 +34,26 @@ BASE_URL='http://localhost:8080' HYDRA_ADMIN_BASE_URL='http://hydra:4445' DSN_REMOTE_DATABASE="pgsql:host='postgres';port=5432;dbname=lasql" APP_LOCALES="fr,en" -NEW_HASH_ALGO="argon2id" HASH_ALGO_LEGACY="sha256, bcrypt" SECURITY_PATTERN="password,salt,pepper" PEPPER= ``` ## Tests password +Dans le cas où plusieurs méthodes de hashage cohabitent (migration de méthode par exemple dans l'application principale), toutes les méthode seront testées pour comparer les hashs jusqu'à un succès -Indication des algorythmes de hashage obsolètes (legacy) sous forme d'une string séparé par des virgules -Indication du nouvel algorythme à utliser -Au login, vérification du mot de passe, et si méthode de hashage obsolète, update du hashage avec nouvelle méthode et requete d'update à la bdd distante. +## Pattern de hashage +``` +Définir le pattern utilisé avec les mots clés autorisé: password | salt | pepper dans un string, séparé par des virgules pour représenter la séquence à employer pour le hashage du mot de passe. +``` ### Postgres ``` -Les mot de passe inscrits en bdd sont hachés en tenant compte du salt si non vide (cf données de la bdd plus bas) et du pepper inscrit en variable d'environnement (généré avec : bin2hex(random_bytes(32)) +Les mot de passe inscrits en bdd sont hachés en tenant compte du salt si non vide (cf données de la bdd plus bas) et du pepper inscrit en variable d'environnement (généré avec : bin2hex(random_bytes(32) pour l'exemple), le pepper peut être vide +Indiquer le nom de la colonne contenant le salt: Il faut inscrire dans slq_login_configuration salt_column_name: salt +Indiquer le pepper utilisé: et conserver le pepper dans service.yaml env(PEPPER): "257d62c24cd352c21b51c26dba678c8ff05011a89022aec106185bf67c69aa8b" @@ -78,7 +81,7 @@ DSN_REMOTE_DATABASE="mysql:host=mariadb;port=3306;dbname=lasql;" |test1@test.com| 8ad4025044b77ae6a5e3fcf99e53e44b15db9a4ecf468be21cbc6b9fbdae6d9f| cesaltestunexemple| Locke|John| |test2@test.com| 50626fa21f45a275cea0efff13ff78fd02234cade322da08b7191c7e9150141d| NULL| Dubois| Angela| |test3@test.com| 504ae1c3e2f5fdaf41f868164dabcef21e17059f5f388b452718a1ce92692c67| cesaltestunautreexemple| Dupont| Henri| - |test4@test.com| $2a$12$zFN0VJ..Cuu.2itWQwmHJe5EUhNHazbMfCSJFpNiEfdwpLzjjDM0u| NULL| Durand|Isabelle| + |test4@test.com| $2a$12$91AHN7WFXieeadvUfZ88mO.9N7oS5adeXbdERnRno9oLAbqqDW4IG| NULL| Durand|Isabelle| A noter que le hash de test4 est en bcrypt ``` diff --git a/src/SQLLogin/SQLLoginRequest.php b/src/SQLLogin/SQLLoginRequest.php index 32c8b3d..d3246a3 100644 --- a/src/SQLLogin/SQLLoginRequest.php +++ b/src/SQLLogin/SQLLoginRequest.php @@ -88,15 +88,4 @@ class SQLLoginRequest return 'SELECT '.$fields.' FROM '.$this->getTableName().' WHERE '.$this->getLoginColumnName().' = :'.$this->getLoginColumnName().';'; } - - public function getRequestUpdatePassword() - { - $fieldsToUpdate = $this->getPasswordColumnName().'= :'.$this->getPasswordColumnName().','; - if (!empty($this->getSaltColumnName())) { - $fieldsToUpdate .= $this->getSaltColumnName().'= :'.$this->getSaltColumnName().','; - } - $fieldsToUpdate = substr($fieldsToUpdate, 0, -1); - - return 'UPDATE '.$this->getTableName().' SET '.$fieldsToUpdate.' WHERE '.$this->getLoginColumnName().' = :'.$this->getLoginColumnName().';'; - } } diff --git a/src/Security/Hasher/PasswordEncoder.php b/src/Security/Hasher/PasswordEncoder.php index dde782d..7512d2e 100644 --- a/src/Security/Hasher/PasswordEncoder.php +++ b/src/Security/Hasher/PasswordEncoder.php @@ -2,7 +2,6 @@ namespace App\Security\Hasher; -use App\SQLLogin\Exception\InvalidSQLLoginAlgoException; use App\SQLLogin\Exception\InvalidSQLLoginConfigurationException; use App\SQLLogin\Exception\InvalidSQLPasswordException; use Symfony\Component\PasswordHasher\Exception\InvalidPasswordException; @@ -18,31 +17,25 @@ class PasswordEncoder implements LegacyPasswordHasherInterface protected ?string $pepper; protected array $hashAlgoLegacy; - protected ?string $newHashAlgo; protected array $securityPattern; - public function __construct(?string $pepper, string $hashAlgoLegacy, ?string $newHashAlgo, string $securityPattern) + public function __construct(?string $pepper, string $hashAlgoLegacy, string $securityPattern) { $this->pepper = $pepper; $this->hashAlgoLegacy = explode(',', $hashAlgoLegacy); - $this->newHashAlgo = $newHashAlgo; $this->securityPattern = explode(',', $securityPattern); } /** - * Utilise l'algo legacy + * Pas utilisé */ public function hash(string $plainPassword, string $salt = null): string { if ($this->isPasswordTooLong($plainPassword)) { throw new InvalidPasswordException(); } - $completedPlainPassword = $this->getPasswordToHash($plainPassword, $salt); - if ($this->isObsoleteAlgo($this->newHashAlgo)) { - throw new InvalidSQLLoginAlgoException(); - } - return password_hash($completedPlainPassword, $this->getValidALgo($this->newHashAlgo)); + return hash($plainPassword.$salt, $this->hashAlgoLegacy[0]); } public function verify(string $hashedPassword, string $plainPassword, string $salt = null): bool @@ -51,80 +44,27 @@ class PasswordEncoder implements LegacyPasswordHasherInterface return false; } - $isNewest = password_get_info($hashedPassword)['algo'] === $this->newHashAlgo; - $completedPassword = $this->getPasswordToHash($plainPassword, $salt); - if ($isNewest) { - if (password_verify($completedPassword, $hashedPassword)) { - return true; - } else { - throw new InvalidSQLPasswordException(); - } - } - // Si le mot de passe a besoin d'un rehash ou qu'il n'y a pas de nouvelle méthode indiquée, on sait qu'il faut utiliser l'une des méthodes legacy pour vérifier le mot de passe - if ($this->needsRehash($hashedPassword) || empty($this->newHashAlgo)) { - foreach ($this->hashAlgoLegacy as $algo) { - if ($this->isObsoleteAlgo($algo)) { - if (hash_equals(hash($algo, $completedPassword), $hashedPassword)) { - return true; - } - } else { - if (password_verify($completedPassword, $hashedPassword)) { - return true; - } + foreach ($this->hashAlgoLegacy as $algo) { + if ($this->isObsoleteAlgo($algo)) { + if (hash_equals(hash($algo, $completedPassword), $hashedPassword)) { + return true; + } + } else { + if (password_verify($completedPassword, $hashedPassword)) { + return true; } - // On vérifie si la méthode legacy est obsolète, si oui on ne peut pas utiliser password_verify, on doit hasher et comparer } - // si on on n'a pas encore retourné de résultat, le mot de passe doit être incorrect - throw new InvalidSQLPasswordException(); - } - // Si on n'est pas rentré dans les conditions précedéntes, on peut utiliser password_verify - if (password_verify($completedPassword, $hashedPassword)) { - return true; - } else { - throw new InvalidSQLPasswordException(); } + throw new InvalidSQLPasswordException(); } public function needsRehash(string $hashedPassword): bool { - // Il y a besoin de tester si on veut mettre à jour le hashage de mot de passe uniquement si le nouveau algo de hashage est indiqué et qu'il est moderne (BCRYPT, ARGON, ...) - if (!empty($this->newHashAlgo) && !$this->isObsoleteAlgo($this->newHashAlgo)) { - return password_needs_rehash($hashedPassword, $this->getValidAlgo($this->newHashAlgo)); - } - return false; } - /** - * @param mixed $algo - * - * @return string - */ - public function getValidAlgo($algo) - { - if ($algo) { - if (in_array($algo, hash_algos())) { - return $algo; - } - $informalAlgos = [ - 'default' => PASSWORD_DEFAULT, - 'bcrypt' => PASSWORD_BCRYPT, - 'argon2i' => PASSWORD_ARGON2I, - 'argon2id' => PASSWORD_ARGON2ID, - ]; - if (in_array($algo, array_keys($informalAlgos))) { - return $informalAlgos[$algo]; - } - if (in_array($algo, password_algos())) { - return $algo; - } - - throw new InvalidSQLLoginAlgoException('Invalid Algorythme'); - } - } - public function isObsoleteAlgo($algo): bool { return in_array($algo, hash_algos()); diff --git a/src/Security/SQLLoginUserAuthenticator.php b/src/Security/SQLLoginUserAuthenticator.php index 4600adc..05d87b3 100644 --- a/src/Security/SQLLoginUserAuthenticator.php +++ b/src/Security/SQLLoginUserAuthenticator.php @@ -82,10 +82,10 @@ class SQLLoginUserAuthenticator extends AbstractAuthenticator // Comparaison remote hash et hash du input password + salt // dump($remoteHashedPassword, $plaintextPassword, $remoteSalt, password_verify($plaintextPassword, $remoteHashedPassword)); $this->passwordHasher->verify($remoteHashedPassword, $plaintextPassword, $remoteSalt); - if ($this->passwordHasher->needsRehash($remoteHashedPassword)) { - $hash = $this->passwordHasher->hash($plaintextPassword); - $this->sqlLoginService->updatePassword($login, $hash, null); - } + // if ($this->passwordHasher->needsRehash($remoteHashedPassword)) { + // $hash = $this->passwordHasher->hash($plaintextPassword); + // $this->sqlLoginService->updatePassword($login, $hash, null); + // } $attributes = $this->sqlLoginService->fetchDatas($login); $user = new User($login, $remoteHashedPassword, $attributes, $rememberMe); diff --git a/src/Service/SQLLoginService.php b/src/Service/SQLLoginService.php index 843e589..db1c957 100644 --- a/src/Service/SQLLoginService.php +++ b/src/Service/SQLLoginService.php @@ -63,23 +63,6 @@ class SQLLoginService extends AbstractController return false; } - public function updatePassword($login, $hashedPassword, $salt) - { - try { - $dbh = $this->getConnection(); - $request = $this->sqlLoginRequest->getRequestUpdatePassword(); - $query = $dbh->prepare($request); - $query->execute([ - $this->sqlLoginRequest->getLoginColumnName() => $login, - $this->sqlLoginRequest->getPasswordColumnName() => $hashedPassword, - $this->sqlLoginRequest->getSaltColumnName() => $salt, - ]); - } catch (PDOException $e) { - \Sentry\captureException($e); - throw new PDOException(); - } - } - public function getConnection() { // Appel du singleton