Object Error object in registration with passport in express

I use Passport and Express in a NodeJs project. I have a User model with fields: id, password and email. When I try to register, this causes this error:

[object of object]

into the form and does not publish user data in the database. The console displays

POST /signup 302 -58.

Here's the whole passport .js file:

var LocalStrategy    = require('passport-local').Strategy;

// load up the user model
var configDB = require('./database.js');
var Sequelize = require('sequelize');
var sequelize = new Sequelize(configDB.url);

var User       = sequelize.import('../app/models/users');
User.sync();

// load the auth variables
var configAuth = require('./auth'); // use this one for testing

module.exports = function(passport) {

    // used to serialize the user for the session
    passport.serializeUser(function(user, done) {
        done(null, user.id);
    });

    // used to deserialize the user
    passport.deserializeUser(function(id, done) {
        User.findById(id).then(function(user){
            done(null, user);
        }).catch(function(e){
            done(e, false);
        });
        });
=========================================================================
        // LOCAL LOGIN =============================================================
    passport.use('local-login', new LocalStrategy({
        // by default, local strategy uses username and password, we will override with email
        usernameField : 'email',
        passwordField : 'password',
        passReqToCallback : true // allows us to pass in the req from our route (lets us check if a user is logged in or not)
    },
    function(req, email, password, done) {      
            User.findOne({ where: { email: email }})
                .then(function(user) {
                    if (!user) {
                        done(null, false, req.flash('loginMessage', 'Unknown user'));
                    } else if (!user.validPassword(password)) {
                        done(null, false, req.flash('loginMessage', 'Wrong password'));
                    } else {
                        done(null, user);
                    }
                })
                .catch(function(e) { 
                    done(null, false, req.flash('loginMessage',e.name + " " + e.message));
                });             
    }));

    =========================================================================
    // LOCAL SIGNUP ============================================================
    passport.use('local-signup', new LocalStrategy({
        // by default, local strategy uses username and password, we will override with email
        usernameField : 'email',
        passwordField : 'password',
        passReqToCallback : true // allows us to pass in the req from our route (lets us check if a user is logged in or not)
    },
    function(req, email, password, done) {        
        //  Whether we're signing up or connecting an account, we'll need
        //  to know if the email address is in use.

        User.findOne({ where: { email: email }})
            .then(function(existingUser) {

                // check to see if there already a user with that email
                if (existingUser) 
                    return done(null, false, req.flash('error', 'That email is already taken.'));

                //  If we're logged in, we're connecting a new local account.
                if(req.user) {
                    var user            = req.user;
                    user.email    = email;
                    user.password = User.generateHash(password);
                    user.save().catch(function (err) {
                        throw err;
                    }).then (function() {
                        done(null, user);
                    });
                } 
                //  We're not logged in, so we're creating a brand new user.
                else {
                    // create the user
                    var newUser = User.build ({email: email, password: User.generateHash(password)}); 
                    newUser.save().then(function() {done (null, newUser);}).catch(function(err) { done(null, false, req.flash('error', err));});
                }
            })
            .catch(function (e) {
                done(null, false, req.flash('loginMessage',e.name + " " + e.message));              
            })

    }));

And in routes.js

// locally --------------------------------
    // LOGIN ===============================
    // show the login form
    app.get('/login', function(req, res) {
        res.render('login.ejs', { message: req.flash('loginMessage') });
    });

    // process the login form
    app.post('/login', passport.authenticate('local-login', {
        successRedirect : '/profile', // redirect to the secure profile section
        failureRedirect : '/login', // redirect back to the signup page if there is an error
        failureFlash : true // allow flash messages
    }));

    // SIGNUP =================================
    // show the signup form
    app.get('/signup', function(req, res) {
        res.render('signup.ejs', { message: req.flash('loginMessage') });
    });

    // process the signup form
    app.post('/signup', passport.authenticate('local-signup', {
        successRedirect : '/profile', // redirect to the secure profile section
        failureRedirect : '/signup', // redirect back to the signup page if there is an error
        failureFlash : true // allow flash messages
    }));

Thanks in advance.

+6
source share
1 answer

I found a solution !!! It was easier than I thought, just negligence. In the user model, I defined a password field:

password: {
      type: DataTypes.STRING(25),
      allowNull: true,
      defaultValue: ''
    },

And after encryption, this field length was too small to hold the value. So I changed it to 255.

+4
source

Source: https://habr.com/ru/post/1016839/


All Articles