ObjectParameterError: Параметр «фильтр» для findOne() должен быть объектом, полученным

Я пытаюсь связать свою страницу входа с мангустом. К сожалению, это не работает. Я получаю сообщение об ошибке:

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

Я пытался найти решение целый день, но я также все еще изучаю кодирование и изо всех сил пытаюсь понять все это, но большую часть я понимаю.

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);
    });
  });

Но я не уверен...

Большое спасибо за вашу помощь.

Пожалуйста, обрежьте свой код, чтобы было легче найти вашу проблему. Следуйте этим рекомендациям, чтобы создать минимально воспроизводимый пример.

Community 19.11.2022 14:25
Создайте ползком, похожим на звездные войны, с помощью CSS и Javascript
Создайте ползком, похожим на звездные войны, с помощью CSS и Javascript
Если вы веб-разработчик (или хотите им стать), то вы наверняка гик и вам нравятся "Звездные войны". А как бы вы хотели, чтобы фоном для вашего...
учебник по Javascript
учебник по Javascript
JavaScript - это язык программирования, который обычно используется для добавления интерактивности и других динамических функций на веб-сайты. Он...
Как использовать d3.js для рисования 2D SVG-элементов в приложении Angular?
Как использовать d3.js для рисования 2D SVG-элементов в приложении Angular?
D3.js - это обширная библиотека, используемая для привязки произвольных данных к объектной модели документа (DOM). Мы разберем основные варианты...
Освоение принципов SOLID в JavaScript: Пошаговое руководство
Освоение принципов SOLID в JavaScript: Пошаговое руководство
Принцип единой ответственности (SRP): класс должен иметь только одну причину для изменения. Другими словами, у него должна быть только одна...
JavaScript - For Loop
JavaScript - For Loop
Наиболее используемая компактная форма оператора цикла.
Неделя 1 - Карточки с временной шкалой
Неделя 1 - Карточки с временной шкалой
Поскольку это была первая неделя, я решил начать с простого. Предполагалось, что у меня будет временная шкала с несколькими карточками, которые можно...
0
1
119
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Вы смешиваете findOne и findById

findById не нужен объект, а _id. Как поясняется в документах (а также в названии метода), под хуком он находит по идентификатору, так и должно быть findOne({_id: id}).

Из документов:

findById(id) почти* эквивалентно findOne({ _id: id })

Итак, проблема в том, что вы пытаетесь использовать просто _id, когда метод ожидает, что объект будет соответствовать фильтру.

Так что вы можете использовать User.findOne({_id: _id}) или User.findById(_id)

Другие вопросы по теме

Использование сервисной учетной записи и ключа JSON, который отправляется вам для загрузки данных в облачное хранилище Google.
Функции входа в систему Laravel
Laravel - Как я могу установить Auth::attempt для пользователя, если у меня есть только токен JWT с адресом электронной почты и идентификатором
ASP.NET Core Web API и Azure: ошибка проверки подлинности, токен доступа в заголовке
Mysql.connector.errors.NotSupportedError: плагин аутентификации «mysql_native_password» не поддерживается только с pyinstaller exe
Как запретить пользователям изменять внешние ключи в запросах?
Аутентификация с реагирующим контекстом и проблема с реагирующим маршрутизатором v6
Анонимные пользователи с токенами утверждений в ядре ASP.NET?
Разделите комбинированное имя пользователя и пароль, используемые в AUTH PLAIN, для использования в AUHT LOGIN для использования с SMTP
Разве SecurityContextHolder не является компонентом?