Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Password Mapper #12

Open
bevsl opened this issue Nov 27, 2015 · 0 comments
Open

Password Mapper #12

bevsl opened this issue Nov 27, 2015 · 0 comments

Comments

@bevsl
Copy link

bevsl commented Nov 27, 2015

Hello goalio,

Thank you for your forgot password module and this doctrine orm adaptation.

I just tested the reset password process with an expired request but it still worked and the password were reseted to passed value.

I investigated a little in your code and I saw the method cleanExpiredForgotRequests($expiryTime) into the password mapper is empty and the same method into the password service do not get the expiryTime from options (TODO commentary).

Have a nice day !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant