YesWiki / yeswiki

YesWiki is a wiki system written in PHP, including extensions making collaboration more simple (databases, maps, easy editing, bootstrap themes,...).
https://yeswiki.net
GNU Affero General Public License v3.0
107 stars 55 forks source link

Sécurisation de la création de comptes #825

Closed RomainLalande closed 2 years ago

RomainLalande commented 3 years ago

Type of issue (keep only one) / Type de demande (ne garder qu'une ligne) Nouvelle fonctionnalité

Description Pour sécuriser la création et gestion des comptes :

J9rem commented 3 years ago

Cette issue est en lien avec #125

VeveQNV commented 2 years ago

Bonjour, je suis en train d'implémenter pour mes besoins la validation de l'inscription par l'envoi d'un mail avec un lien d'activation. J'utilise la table triples pour stocker les informations liées à ce processus (clefs et booleen sur l'activation du compte). Si le code vous intéresse je vous le proposerais.

J9rem commented 2 years ago

@VeveQNV il y a déjà des travaux en cours avec #935 qui va bientôt être fusionnée. Nous pourrons alors repartir de la nouvelle version de YesWiki pour voir comment mettre en place les besoins de validation (je serai alors curieux de voir comment tu proposes de faire à partir de la future nouvell eversion)

VeveQNV commented 2 years ago

Oui j'ai vu cela. A la fois, je n'ai aucune visibilité sur la sortie de tout ceci. Néanmoins je pense que la restructuration du code est un détail en 1 sens. L'algorithme a proprement parlé peut s'intégrer facilement ici ou là, A peu de choses près, c'est le nom des fonctions qui changent.

Niveau du fonctionnement : je rajoute un triple qui indique si le compte est activé ou non. A la fin de la création de la fiche j'envoie un mail avec un lien contenant une clef que j'ai stocké également dans la table triples. Puis je redirige vers une page qui dit qu'il faut valider le compte en cliquant sur le lien. Je rajoute une action qui est appelé à partir de la page du lien. Elle valide le compte utilisateur si la clef est bonne. Au login on teste si le compte est activé.

Le 08/07/2022 à 22:59, Jérémy Dufraisse a écrit :

@VeveQNV https://github.com/VeveQNV il y a déjà des travaux en cours avec #935 https://github.com/YesWiki/yeswiki/pull/935 qui va bientôt être fusionnée. Nous pourrons alors repartir de la nouvelle version de YesWiki pour voir comment mettre en place les besoins de validation (je serai alors curieux de voir comment tu proposes de faire à partir de la future nouvell eversion)

— Reply to this email directly, view it on GitHub https://github.com/YesWiki/yeswiki/issues/825#issuecomment-1179356156, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKLWGR5ELEUD4RM7H33YOETVTCJEPANCNFSM5FRRJD6A. You are receiving this because you were mentioned.Message ID: @.***>

J9rem commented 2 years ago

fixed by #935