Hi Kelvin, great article, thanks. I just spotted that you forgot to check whether reset token is valid or not in '/reset-confirm/:token' Post method.
'/reset-confirm/:token'
const passwordReset = await PasswordReset.findOne({ token })
passwordReset is not being checked after this line.
passwordReset
Hi Mete,
Good eye. If we don't check it here, the user update will throw an error. To avoid this we can add a guard clause to check the password reset object:
if (!passwordReset) { return res.status(404).send() } // Continue with the reset
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Hi Kelvin, great article, thanks. I just spotted that you forgot to check whether reset token is valid or not in
'/reset-confirm/:token'
Post method.passwordReset
is not being checked after this line.Hi Mete,
Good eye. If we don't check it here, the user update will throw an error. To avoid this we can add a guard clause to check the password reset object: