FullstackAcademy / boilermaker

Code scaffold for projects

Home Page:https://www.youtube.com/watch?v=7bLSuTHH4Ag&list=PLx0iOsdUOUmn7D5XL4mRUftn8hvAJGs8H

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Restore missing error handling for user login

glebec opened this issue · comments

router.post('/login', async (req, res, next) => {
const user = await User.findOne({where: {email: req.body.email}})
if (!user) {
console.log('No such user found:', req.body.email)
res.status(401).send('Wrong username and/or password')
} else if (!user.correctPassword(req.body.password)) {
console.log('Incorrect password for user:', req.body.email)
res.status(401).send('Wrong username and/or password')
} else {
req.login(user, err => (err ? next(err) : res.json(user)))
}
})

During #20 we accidentally forgot to include error handling for failed login attempts. We should put that back.

bitmoji