MineWeb / MineWebCMS

🚀 A French Minecraft CMS since 2015 (used by +1k websites)
https://mineweb.org
GNU General Public License v3.0
93 stars 41 forks source link

improv. auth perf #262

Closed Arkadax closed 3 years ago

Arkadax commented 3 years ago

Bonjour J'ai amélioré les performances lors de l'inscription des users en enlevant les hash inutiles comme le deuxieme hash a l’inscription et le hash si un utilisateur n'existe pas. J'ai aussi diminué le nombre de requêtes mysql J'ai tout testé plusieurs fois et tout marche correctement Je m'excuse pour le formattage comme il est pas consistent sur tout les fichiers et il y a beaucoup de lignes très longues, j'ai pas pu configurer mon éditeur. J'ai fait ces modifications il y a plusieurs mois pour mon serveur car on avait constaté que cette partie utilisait beaucoup le serveur lors de tests de charges. Comme MineWeb redevient actif je me suis dis que c'est mieux d'en faire profiter tout le monde plutot que de garder ca pour moi et le serveur a fermé alors ca me sert a rien J'en ai profité pour réparer le sha386 qui n'existe pas et qui fait que tous les mots de passes sont vides quand il est utilisé ce qui est très dangereux car tout le monde peut se login sur tous les comptes !!!!! Je dev énormément avec symfony mais je ne fais jamais de contribution sur github, si j'ai fait des erreurs dans cette proposition dites moi car je voudrais après faire d'autres améliorations :wink: Bonne soirée

nivcoo commented 3 years ago

Et ensuite pour le sha, bien vu

nivcoo commented 3 years ago

Je viens de regarder pour le pl Auth, il utilise pas donc c'est OK pour ça mais faudrait changer les fichiers que tu n'utilise pas et corrigé le problème de cpatcha et ce seront bon

Arkadax commented 3 years ago

C'est good j'ai fait les changements

nivcoo commented 3 years ago

C'est good pour moi je pense