不确定此express-session auth方法是否安全。我使用bcryptjs
加密密码,加密后在req.session.isLoggedIn = true
中设置admin.js
,随后使用if语句在events.js
中对其进行检查。 events.js
中的if语句方法安全吗?有更好的选择吗?
我正在使用把手来渲染网页。
admin.js
bcrypt.compare(pass, user.password).then((doMatch) => {
console.log(doMatch);
//Check if password match
if (doMatch) {
//setting the isLoggedIn value
req.session.isLoggedIn = true;
//Events is the route that requires authentication
return res.redirect('/events');
} else {
res.redirect('/');
}
}).catch((err) => {
console.log(err);
});
events.js
Router.get('/', (req, res) => {
//Checking the if the loggedIn value is true
if (req.session.isLoggedIn) {
Event.find({}, (err, events) => {
res.render('events', {
prods: events,
pageTitle: 'Events',
path: '/events',
hasProducts: events.length > 0
});
}).catch((err) => {
console.log(err);
});
} else {
console.log('User not authenticated');
res.status(404).send({error: 'not authorized!'});
}
});
答案 0 :(得分:0)
我认为这不是您应该查看的最佳方法:http://www.passportjs.org/
他们有关于身份验证策略和注册的良好文档,还有许多其他方法(Facebook登录,Twitter ...),还有许多关于如何实现Passport.js的教程
希望它对您有帮助!