Chiffrer les mots de passe en base de donnée

[Sebbaz]

Bonjour,

Actuellement, les mots de passe sont stockés en clair dans la base de donnée quand on part d'une installation depuis le projet sur la branche master.

Il est important que les mots de passe soit mieux protégés.

Symfony implémente de base des bibliothèques de sécurité permettant de protéger les informations sensibles à l'aide d'un chiffrement asymétrique. https://symfony.com/doc/current/configuration/secrets.html

C'est prévu ? Le projet est ouvert aux contributions ?

[abouchereau]

Bonjour,

Effectivement c'est quelque chose que j'aimerais faire.
Il faudrait que cela s'accompagne d'autres fonctionnalités :

• une page Profil ou chaque utilisateur peut modifier son mot de passe (en plus d'autres informations : nom, adresse mail etc...)
• sur la page d'authentification un lien "J'ai oublié mon mot de passe" avec le fonctionnement traditionnel : envoi de mail avec un lien temporaire pour modifier le mot de passe etc...

Il n'y a pas encore de contribution externe sur le projet mais c'est tout à fait envisageable.

[Sebbaz]

Je pense avoir un peu de temps ce week-end, je vais regarder tout ça pour proposer une PR

[Sebbaz]

J'ai ajouté une première partie des developpements dans le commit eaaa05a mais je n'ai pas vérifié la partie pour mettre à jour le mot de passe depuis le formulaire d'administration des adhérents.

Il reste aussi à rajouter les fonctionnalités suivantes :

• page de profil
• réinitialisation de mot de passe

Le code est à jour avec la branche master, s'il faut le revoir par rapport à une autre branche n'hésitez pas à demander :)

[Sebbaz]

Bonjour,

Je rencontre quelques difficultés pour ajouter un formulaire de réinitiation de mot de passe avec SymfonyCasts/reset-password-bundle : peut-être auriez vous des conseils ?

SymfonyCasts/reset-password-bundle demande d'éxécuter une migration doctrine php bin/console make:migration à la suite de l'utilisation de l'utilitaire de génération de code. Sauf que la migration doctrine renvoie un message d'erreur et ce même si on efface complètement la base de donnée :

$ php7 bin/console make:migration

In SchemaException.php line 112:
                                                                       
  The table with name 'easyamaptest.contract_product' already exists.  
                                                                       

make:migration [--db DB] [--em [EM]] [--shard SHARD] [-h|--help] [-q|--quiet] [-v|vv|vvv|--verbose] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--] <command>

J'ai l'impression que les migrations doctrine ne fonctionnent pas sur le sujet, il vaut mieux passer par les scripts SQL édités sous https://github.com/abouchereau/easyamap/tree/master/data ?

[Sebbaz]

Autrement pour utiliser SymfonyCasts/reset-password-bundle, il faut utiliser une version de symfony/flex ^1.6 sans quoi php bin/console make:reset-password plante complètement le projet à cause d'un problème d'injection.

Sur la branche master on est en version 1.4 (

easyamap/composer.json

Line 12 in f026a55

"symfony/flex": "^1.4",

) : pas objections à passer en ^1.6 ?

[abouchereau]

Pas d'objection :)
Merci à toi de regarder ça.
Idéalement ce serait mieux sur une branche à part effectivement.

[Sebbaz]

Voilà, j'ai tout mis dans https://github.com/Sebbaz/easyamap/tree/dev-issue-2-encodepassword

Il reste à modifier l'email qui part quand on crée l'utilisateur dans l'interface d'admin. L'email fait référence à un mot de passe vide. Je serais d'avis de générer un mot de passe par défaut et d'inviter l'utilisateur à utiliser le formulaire de réinitialisation de mot de passe pour mettre un mot de passe à lui.

La page d'authentification a un bouton pour utiliser un lien vers la réinitalisation de mot de passe et ensuite il y a une série d'écrans. Il y a deux contraintes lors de la réinitialisation :

• taper deux fois le même mot de passe
• saisir un mot de passe assez long - 6 caractères - pas paramétrable en dur dans le code mais c'est simple à modifier pour un dev ⛏️ (c'est en TODO pour ajouter ça en conf')

https://wtf.roflcopter.fr/pics/gallery#EQAYxZNd/ZKM7LVc2.jpg,poe1HJPa/926MbDLd.jpg,GRznzbLQ/jG5iffqE.jpg,pnVvemct/tNarPAq6.jpg,CQj3y01e/5Mis4glL.jpg,BBtrNNgU/btmn33la.jpg,jPiqKeeu/hbXVRT5G.jpg

Je me suis fait plaisir en modifiant les couleurs de boutons pour présenter le formulaire en respectant la l'économie d'attention qu'on peut gagner avec les méchants darkpatterns. Je part sur l'hypothèse qu'on a plus tendance à s'identifier qu'à réinitialiser un mot de passe. Si vous trouvez ça moche n'hésitez pas à moinsoyer :)

Après il ne restera plus que la page de profil !

[Sebbaz]

Petites précisions en plus :

J'ai modifié le fichier de traduction config/packages/translation.yaml pour mettre en 'fr' par défaut la traduction sans quoi une partie des messages ne sont pas traduits de l'anglais vers le français

J'ai abandonné les migrations doctrines vu que ça ne marche pas en l'état dans le projet. le script de création de la table associée est ici : https://github.com/Sebbaz/easyamap/blob/dev-issue-2-encodepassword/data/password_feature.sql