Postgres results.rows [0] не определены - PullRequest
0 голосов
/ 22 февраля 2019

Не уверен, что я делаю неправильно, кроме написания действительно грязного кода для проекта, который я делаю, чтобы выучить Nodejs.

Раньше это была асинхронная функция / объект, но решил избавиться отпопробуйте catch, потому что мой код запускался дважды по какой-то причине, которую я не мог понять.

Устранение try catch не помешало ему по-прежнему работать дважды, я думаю.

Итак, вопрос: почему мои results.rows [0] .email возвращаются как неопределенные?

Иногда это работает, иногда нет.Я не знаю почему.Любая помощь будет качаться.

   router.post('/', (req, res, next) => {
    const {password, email} = req.body
            //var LoginPwd = await bcrypt.hash(password, 5);
    const loginPlainPwd = password;

    pool.query("SELECT password, id, email FROM companies_admins WHERE email=$1", [email], (err, results) => {
        if (err)
        {
            throw err;
        }

        const dbemail = results.rows[0].email
        const dbPwd = results.rows[0].password 
        const dbid = JSON.stringify(results.rows[0].id)
        console.log('results.rows[0] = ' + results.rows[0])
        console.log('loginPlainPwd = ' + loginPlainPwd)
        console.log('dbPwd = ' + dbPwd)
        //console.log(JSON.stringify(results.rows[0]))
        //res.cookie('userId', id)
        //res.sendFile(path.join(__dirname, './views/account.html'));
        //bcrypt.compare(loginPlainPwd, dbPwd, (err, res) => {
        if (loginPlainPwd != dbPwd) 
        {
            console.log("loginPlainPwd != dbPwd")
            /////////////////////////////////////////////?SHOULD THIS BE OUTSIE POOL.QUERY??????
            console.log('err')
            return res.status(401).json({
                message: 'Auth failed'
            });
        }
        else if (loginPlainPwd == dbPwd) 
        {
            //token variable signage/creation with user data and expiration (i also included .env)
            const token = jwt.sign(
                {
                    email: dbemail,
                    userId: dbid,
                }, 
                process.env.JWT_KEY, 
                {
                    expiresIn: "1h"
                },
            );

            console.log("passwords match: token created:" + token)
            res.cookie('userId', token,)

            console.log('cookie should be sent')
            databaseJWTin(err, token, dbemail); // database function to store jwttoken from below to store jwt in database
            console.log('databaseJWT function should have fired')
            //had to use ../ below because path was going into routes directory for some reason
            res.sendFile(path.join(__dirname, '../views/account.html'))
            //return res.status(200).json({
            //  message: "Auth successful",
            //  token: token
            //});
        }
        //res.sendFile(path.join(__dirname, './views/account.html'))
    });
    //res.sendFile(path.join(__dirname, './views/account.html'));
})

1 Ответ

0 голосов
/ 22 февраля 2019

Пожалуйста, проверьте, содержит ли результат данные.

router.post('/', (req, res, next) => {
    const { password, email } = req.body
    //var LoginPwd = await bcrypt.hash(password, 5);
    const loginPlainPwd = password;

    pool.query("SELECT password, id, email FROM companies_admins WHERE email=$1", [email], (err, results) => {
        if (err) {
            throw err;
        }
        if (results && results.length>0) {
            const dbemail = results.rows[0].email
            const dbPwd = results.rows[0].password
            const dbid = JSON.stringify(results.rows[0].id)
            console.log('results.rows[0] = ' + results.rows[0])
            console.log('loginPlainPwd = ' + loginPlainPwd)
            console.log('dbPwd = ' + dbPwd)
            //console.log(JSON.stringify(results.rows[0]))
            //res.cookie('userId', id)
            //res.sendFile(path.join(__dirname, './views/account.html'));
            //bcrypt.compare(loginPlainPwd, dbPwd, (err, res) => {
            if (loginPlainPwd != dbPwd) {
                console.log("loginPlainPwd != dbPwd")
                /////////////////////////////////////////////?SHOULD THIS BE OUTSIE POOL.QUERY??????
                console.log('err')
                return res.status(401).json({
                    message: 'Auth failed'
                });
            }
            else if (loginPlainPwd == dbPwd) {
                //token variable signage/creation with user data and expiration (i also included .env)
                const token = jwt.sign(
                    {
                        email: dbemail,
                        userId: dbid,
                    },
                    process.env.JWT_KEY,
                    {
                        expiresIn: "1h"
                    },
                );

                console.log("passwords match: token created:" + token)
                res.cookie('userId', token)

                console.log('cookie should be sent')
                databaseJWTin(err, token, dbemail); // database function to store jwttoken from below to store jwt in database
                console.log('databaseJWT function should have fired')
                //had to use ../ below because path was going into routes directory for some reason
                res.sendFile(path.join(__dirname, '../views/account.html'))
                //return res.status(200).json({
                //  message: "Auth successful",
                //  token: token
                //});
            }
            //res.sendFile(path.join(__dirname, './views/account.html'))
        }

    });
    //res.sendFile(path.join(__dirname, './views/account.html'));
})
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...