ObjectParameterError: findOne() 的参数 \"filter\" 必须是一个对象，得到

Question

我尝试将我的登录页面与 mongoose 连接起来。不幸的是它不起作用。 我收到错误：

ObjectParameterError: Parameter "filter" to findOne() must be an object, got 637798b57bfa9d5fbede9c30

我一整天都在努力寻找解决方案，但我 cant figure it out. Im 还在学习编码并努力理解它的一切，但我大部分都理解。

if(process.env.NODE_ENV !== "production") {
    require("dotenv").config()
}

const express = require('express')
const app = express()
const bcrypt = require('bcrypt')
const passport = require("passport")
const initializePassport = require("./passport-config")
const flash = require("express-flash")
const session = require("express-session")
const methodOverride = require("method-override")
const mongoose = require("mongoose")
const User = require("./user")

const uri = 'abc'

async function connect(){
    try{
        await mongoose.connect(uri)
        console.log("connected")
    } catch (error){
        console.error(error)
    }
}

connect();

const user = User

initializePassport(
    passport,
    email => User.find(user => user.email === email),
    _id => User.find(user => user._id === _id)
    )

app.set('view-engine', 'ejs')
app.use(express.urlencoded({extended: false}))
app.use(flash())
app.use(session({
    secret: process.env.SESSION_SECRET,
    resave: false, // We wont resave the session var if nothing is changed
    saveUninitialized: false
}))
app.use(passport.initialize())
app.use(passport.session())
app.use(methodOverride("_method"))

function initialize(passport, getUserByEmail, getUserById) {
    const authenticateUser = async (email, password, done) => {
      const user = await User.findOne({email:email});
      console.log(user);
      if (user == null) {
        return done(null, false, { message: 'No user with that email' })
      }
  
      try {
        if (await bcrypt.compare(password, user.password)) {
          return done(null, user)
        } else {
          return done(null, false, { message: 'Password incorrect' })
        }
      } catch (e) {
        return done(e)
      }
    }
  
    passport.use(new LocalStrategy({ usernameField: 'email' }, authenticateUser))
    passport.serializeUser((user, done) => done(null, user.id))
    passport.deserializeUser((id, done) => {
      return done(null, getUserById(id))
    })
  }
  
  module.exports = initialize

app.get('/', checkAuthenicated, (req, res)=> {
    res.render('index.ejs', {name: req.body.name})
})

app.get('/login', checkNotAuthenicated, (req, res)=> {
    res.render('login.ejs')
})

app.post("/login", checkNotAuthenicated, passport.authenticate("local", {
    successRedirect: "/",
    failureRedirect: "/login",
    failureFlash: true
}))

app.get('/register', checkNotAuthenicated, (req, res)=> {
    res.render('register.ejs')
})

app.post("/register", checkNotAuthenicated, async (req, res) => {
    try {
       const hashedPassword = await bcrypt.hash(req.body.password, 10)
        //users.push({
       const user = new User({
            id: Date.now().toString(),
            username: req.body.name,
            email: req.body.email,
            password: hashedPassword
        })
        user.save().then(()=> console.log("User saved"))
        console.log(user)
        res.redirect("/login")

    } catch (e){
        console.log(e);
        res.redirect("/register")
    }
})

app.delete("/logout", (req, res) =>{
    req.logOut(
        res.redirect("/login")
    )
})


function checkAuthenicated(req, res, next){
    if (req.isAuthenticated()){
        return next()
    }
    res.redirect("/login")
}

function checkNotAuthenicated(req, res, next){
    if (req.isAuthenticated()){
        return res.redirect("/")
    }
    next()
}

app.listen(3000)

const bcrypt = require('bcrypt')
const mongoose = require("mongoose")

const userShema = new mongoose.Schema({
    id: String,
    username: String,
    email: String,
    password: String
})

module.exports = mongoose.model("User", userShema)

const LocalStrategy = require('passport-local').Strategy
const bcrypt = require('bcrypt')
const User = require("./user")

function initialize(passport, getUserByEmail, getUserById) {
  const authenticateUser = async (email, password, done) => {
    const user = await User.findOne({email:email});
    console.log(user);
    if (user == null) {
      return done(null, false, { message: 'No user with that email' })
    }

    try {
      if (await bcrypt.compare(password, user.password)) {
        return done(null, user)
      } else {
        return done(null, false, { message: 'Password incorrect' })
      }
    } catch (e) {
      return done(e)
    }
  }

  passport.use(new LocalStrategy({ usernameField: 'email' }, authenticateUser))
  passport.serializeUser((User, done) => done(null, User._id))
  passport.deserializeUser(function(_id, done) {
    User.findOne(_id, function (err, User) {
      done(err, User);
    });
  });
}

module.exports = initialize

我认为问题出在这里的某个地方：

  passport.use(new LocalStrategy({ usernameField: 'email' }, authenticateUser))
  passport.serializeUser((User, done) => done(null, User._id))
  passport.deserializeUser(function(_id, done) {
    User.findOne(_id, function (err, User) {
      done(err, User);
    });
  });

但我不确定...

非常感谢您的帮助。

Answer 1

您正在混合 findOne 和 findById

findById 不需要对象，但需要 _id。正如文档（以及方法名称）所解释的那样，在它通过 id 找到的钩子下，findOne({_id: id}) 也是如此。

从文档：

findById(id)几乎*等同于findOne({ _id: id })

所以问题是当方法期望对象与过滤器匹配时，您正试图简单地使用 _id。

所以你可以使用User.findOne({_id: _id})或User.findById(_id)