297 password reset wea 8000 k password #145
Conversation
| @@ -0,0 +1,52 @@ | |||
| <?php | |||
|
|
|||
| namespace SumoCoders\FrameworkCoreBundle\Security; | |||
There was a problem hiding this comment.
Eventueel uitleggen waarom je https://github.com/symfony/symfony/blob/7.0/src/Symfony/Component/Validator/Constraints/PasswordStrengthValidator.php niet rechtstreeks gebruikt. Omdat het private static method is?
There was a problem hiding this comment.
Dat lijkt gewoon een kopie van Symfony code? Niet beter om dat een validator te gebruiken in uw Ajax call? Zoals eerste voorbeeld op https://symfony.com/doc/current/components/validator.html (maar dan met PasswordStrength Assert). Want als Symfony hun "strength" update, dan gaan we onze code ook moeten updaten.
There was a problem hiding this comment.
@jonasdekeukelaere
het probleem daarmee is dat je niet de waarde 1 -> 4 terug krijgt. Daphne heeft die nodig voor de frontend
There was a problem hiding this comment.
@tijsverkoyen fixed c1224ba
There was a problem hiding this comment.
Zou het dan geen idee zijn om een PR naar symfony te doen om die waarde wel terug te geven? Al is het maar die method public maken? Als je uitlegt dat het handig kan zijn om password strength indicator te kunnen bouwen?
Alternatief is eventueel de strength in de "message" verwerken zoals bij https://github.com/symfony/symfony/blob/78c6ceb5cd0560346b6765849f142c1c45 8000 1fd1a8/src/Symfony/Component/Validator/Constraints/File.php#L57. Maar oprecht geen idee hoe dat werkt.
Want wat Jonas zegt klopt volledig dat we dat nu zelf gaan moeten onderhouden. Plus het kan ook zijn dat er verschillen optreden op termijn.
Ale, for now mergen, maar ook PR doen. En als die gemerged zou worden dan kunnen we "onze" code schrapen.
There was a problem hiding this comment.
symfony PR aangemaakt symfony/symfony#54405
There was a problem hiding this comment.
Top, er is wel al feedback. Maybe best eens checken.
zie https://app.activecollab.com/108877/reports/team-timeline?modal=Task-195953-533
wordt daarna gebruikt in https://github.com/sumocoders/Framework-User-Implementation-Example/tree/297-password-reset-weak-password