【问题标题】:bcryptjs compare function returns false when passwords contain numbers当密码包含数字时,bcryptjs 比较函数返回 false
【发布时间】:2020-12-04 01:44:08
【问题描述】:

我已使用 bcryptjs 对我的密码进行哈希处理,并且用户注册正常。 但是当我尝试登录时,如果密码包含数字或其他字符,bcrypt.compare 函数会返回一个 false 承诺。
如果密码只是字母,一切正常。
我到处搜索,但找不到解决此错误的方法?
这是我的文件:

users.js(模型)

var mongoose = require("mongoose");
var bcrypt = require("bcryptjs");

var userSchema = mongoose.Schema({
    name: String,
    email: String,
    password: String,
    products: [{
        type: mongoose.Schema.Types.ObjectID,
        ref: "Product"
    }],
    username: String
});

var User = module.exports = mongoose.model("User", userSchema);
module.exports.createUser = function(newUser, callback) {
    bcrypt.genSalt(5,function(err,salt){
        bcrypt.hash(newUser.password,salt,function(err,hash){
            newUser.password = hash;
            newUser.save(callback);
        });
    });
}

module.exports.comparePassword = function(candidatePassword, hash, callback){
    bcrypt.compare(candidatePassword, hash, function(err, isMatch) {
      if(err) throw err;
      console.log(isMatch);
      callback(null, isMatch);
    });
  }
  module.exports.getUserByUsername = function(username, callback){
    var query = {username: username};
    User.findOne(query, callback);
  }

app.js

app.use(passport.initialize());
app.use(passport.session());
var LocalStrategy = require("passport-local").Strategy;
passport.use(new LocalStrategy(
    function(username, password, done) {
        User.findOne({username:username}, function(err, user){
            if(err) throw err;
            if(!user) {
                return done(null,false, {message: 'Unknown user'});
            }
            User.comparePassword(password,user.password,function(err, isMatch){
                if(err) {
                    console.log("ERR1");
                    return done(err);
                }
                if(isMatch) {
                    console.log("MATCH");
                    return done(null,user);
                } else {
                    console.log("NOT VALID");
                    return done(null, false, {message: 'Invalid password'});
                }
            });
        });
    }
));

passport.serializeUser(function(user,done){
    done(null,user.id);
});
passport.deserializeUser(function(id,done){
    User.getUserById(id, function(err,user){
        done(err,user);
    })
})

users.js (ROUTE)

router.post("/login", passport.authenticate('local', function (error, user, info){
    if(error) {
        console.error(error);
            console.log('Failed login:');
    }
    if (user === false) {
        console.log("user not found");
    } else {
        // handle successful login ...
        console.log("logged in");
    }
}), function (req, res) {
    res.send(req.user);
});

router.post("/signup", function (req, res) {
    console.log(req.body);
    var password = req.body.password;
    var password2 = req.body.password2;
    if(password == password2) {
        var newUser = new User ({
            name: req.body.name,
            email: req.body.email,
            username: req.body.username,
            password: req.body.username
        });

        User.createUser(newUser, function(err, user){
            if(err)
                throw err;
            console.log(user);
            res.send(user).end();   
        });
    } else {
        res.status(500).send("{errors: \"Passwords don't match\"}").end()
    }
});

每当我输入包含数字的密码时,我都会得到

false
NOT VALID
user not found

如果我做了或没有做一些非常简单的事情,我很抱歉。这是我第一次使用 bcryptjs。 感谢所有答案!

【问题讨论】:

    标签: javascript node.js express authentication bcrypt


    【解决方案1】:

    在 bcrypt 的 compare 方法内的 comparePassword 函数的 users.js 文件中,您将抛出错误而不是将其传递给回调。也许如果你做出这个改变:

    module.exports.comparePassword = function(candidatePassword, hash, callback){
        bcrypt.compare(candidatePassword, hash, function(err, isMatch) {
          console.log(isMatch);
          callback(err, isMatch);
        });
    }
    

    当您在密码中引入数字时,您会看到案例产生了什么样的错误。

    【讨论】:

    • 非常感谢您的回复。我进行了更改, err 仍然为空。即没有错误,isMatch 仍然为 false。
    猜你喜欢
    • 2015-01-18
    • 2017-02-25
    • 2020-05-24
    • 2017-02-25
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多