【发布时间】：2017-07-29 16:55:15
【问题描述】：
我已经设置了使用 NodeJS/Passport 更新用户密码。 我遵循了这个很棒的指南：http://sahatyalkabov.com/how-to-implement-password-reset-in-nodejs/。
其中 99% 是有效的。我不得不对其进行一些修改以包含一些条带功能。恐怕我在某处有一个严重错误，但我找不到它。用户目前能够一路完成向他们发送电子邮件、输入新密码并登录的过程。随后的另一封电子邮件表明他们的密码已成功更新。都完美。然而。因为某些原因。未保存新密码。用户只能使用旧密码登录。我已经尝试了我能想到的一切来解决这个问题。
我已经让其他几个程序员研究过这个问题，但他们都无法弄清楚它到底是如何不工作的。
当前的想法是会话可能无法正确结束，但我们尝试销毁会话，但仍然无法正常工作。
非常感谢任何帮助。
完整设置：
用户模型：
var UserSchema = new mongoose.Schema({
username: { type: String, required: true, unique: true },
password: String,
datapoint: String,
email: { type: String, required: true, unique: true },
resetPasswordToken: String,
resetPasswordExpires: Date
});
UserSchema.pre('save', function(next) {
var user = this;
var SALT_FACTOR = 5;
if (!user.isModified('password')) return next();
bcrypt.genSalt(SALT_FACTOR, function(err, salt) {
if (err) return next(err);
bcrypt.hash(user.password, salt, null, function(err, hash) {
if (err) return next(err);
user.password = hash;
next();
});
});
});
注册新帐户 （这也包含无关的条带信息，但可能会导致问题。）
var newUser = new User({username: req.body.username, email: req.body.email, datapoint: req.body.datapoint});
User.register(newUser, req.body.password, function(err, user){
if(err){
console.log('Looks like there was an error:' + ' ' + err)
res.redirect('/login')
} else {
passport.authenticate("local")(req, res, function(){
var user = new User({
username: req.body.username,
email: req.body.email,
password: req.body.password
})
console.log('creating new account')
console.log('prepping charge')
var token = req.body.stripeToken; // Using Express
var charge = stripe.charges.create({
amount: 749,
currency: "usd",
description: "Example charge",
source: token,
}, function(err, charge) {
// asynchronously called
console.log('charged')
});
res.redirect('/jobquiz')
console.log(req.body.datapoint)
console.log(req.body.email)
});
}
});
});
设置忘记密码发布
app.post('/forgot', function(req, res, next) {
async.waterfall([
function(done) {
crypto.randomBytes(20, function(err, buf) {
var token = buf.toString('hex');
done(err, token);
});
},
function(token, done) {
User.findOne({ email: req.body.email }, function(err, user) {
if (!user) {
// console.log('error', 'No account with that email address exists.');
req.flash('error', 'No account with that email address exists.');
return res.redirect('/forgot');
}
console.log('step 1')
user.resetPasswordToken = token;
user.resetPasswordExpires = Date.now() + 3600000; // 1 hour
user.save(function(err) {
done(err, token, user);
});
});
},
function(token, user, done) {
console.log('step 2')
var smtpTrans = nodemailer.createTransport({
service: 'Gmail',
auth: {
user: 'myemail',
pass: 'mypassword'
}
});
var mailOptions = {
to: user.email,
from: 'myemail',
subject: 'Node.js Password Reset',
text: 'You are receiving this because you (or someone else) have requested the reset of the password for your account.\n\n' +
'Please click on the following link, or paste this into your browser to complete the process:\n\n' +
'http://' + req.headers.host + '/reset/' + token + '\n\n' +
'If you did not request this, please ignore this email and your password will remain unchanged.\n'
};
console.log('step 3')
smtpTrans.sendMail(mailOptions, function(err) {
req.flash('success', 'An e-mail has been sent to ' + user.email + ' with further instructions.');
console.log('sent')
res.redirect('/forgot');
});
}
], function(err) {
console.log('this err' + ' ' + err)
res.redirect('/');
});
});
app.get('/forgot', function(req, res) {
res.render('forgot', {
User: req.user
});
});
设置更改密码帖子
app.get('/reset/:token', function(req, res) {
User.findOne({ resetPasswordToken: req.params.token, resetPasswordExpires: { $gt: Date.now() } }, function(err, user) {
console.log(user);
if (!user) {
req.flash('error', 'Password reset token is invalid or has expired.');
return res.redirect('/forgot');
}
res.render('reset', {
User: req.user
});
});
});
app.post('/reset/:token', function(req, res) {
async.waterfall([
function(done) {
User.findOne({ resetPasswordToken: req.params.token, resetPasswordExpires: { $gt: Date.now() } }, function(err, user, next) {
if (!user) {
req.flash('error', 'Password reset token is invalid or has expired.');
return res.redirect('back');
}
user.password = req.body.password;
user.resetPasswordToken = undefined;
user.resetPasswordExpires = undefined;
console.log('password' + user.password + 'and the user is' + user)
user.save(function(err) {
if (err) {
console.log('here')
return res.redirect('back');
} else {
console.log('here2')
req.logIn(user, function(err) {
done(err, user);
});
}
});
});
},
function(user, done) {
// console.log('got this far 4')
var smtpTrans = nodemailer.createTransport({
service: 'Gmail',
auth: {
user: 'myemail',
pass: 'mypass'
}
});
var mailOptions = {
to: user.email,
from: 'myemail',
subject: 'Your password has been changed',
text: 'Hello,\n\n' +
' - This is a confirmation that the password for your account ' + user.email + ' has just been changed.\n'
};
smtpTrans.sendMail(mailOptions, function(err) {
// req.flash('success', 'Success! Your password has been changed.');
done(err);
});
}
], function(err) {
res.redirect('/');
});
});
【问题讨论】：
-
我试过你发布的代码没有条纹部分，它对我来说工作正常。当发生这么多事情时，要查明问题要困难得多。隔离电话并一次接一个电话可能会对您有更多帮助，我认为如果您可以提供您认为可能出错的区域，这将有助于有人接听电话。
-
我不确定您是否已经找到解决问题的方法。有几个会话设置可能会导致您的配置中缺少此问题。如果你还没有，我认为它值得研究。考虑用
app.use(session({ secret: 'session secret key', resave: false, saveUninitialized: false })); app.use(passport.initialize()); app.use(passport.session());替换app.use(session({ secret: 'session secret key' }));并验证订单。你可以在这里查看更多关于这些标志的信息github.com/expressjs/session#options -
预存中间件中的所有行都执行了吗？那是我首先开始调试的地方。
-
通常不建议告诉用户不存在具有指定电子邮件地址的用户 - 即使是这样。您应该只说“如果存在使用此电子邮件地址的用户，我们已发送密码重置链接”。没有理由向潜在的攻击者提供更多信息。他们以你的方式拥有它，你让他们能够准确地找出你的用户是谁。
标签： node.js mongodb express passwords passport.js