Почему ругается цикл(js)?

Question

[historydev]

Доброе утро.Вопрос возможно глуп,но я не понимаю,по идее проходит запрос в базу и создает константы с данными затем уже идет цикл,почему он не видит isIds?

ReferenceError: isIds is not defined
at Object. (C:\Users\History\Desktop\session\config.js:65:12)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (C:\Users\History\Desktop\session\app.js:25:1)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

let userDB = null;

sequelize
  .query('SELECT * FROM users', { raw: true })
  .then(users => {
  	const isIds = users[0].map(user => user.id);
  	const isEmails = users[0].map(user => user.email);
  	const isPasswords = users[0].map(user => user.password);

  	userDB = {
		id: isIds,
		email: isEmails,
		password: isPasswords
	}
});

for(i=0; i<isIds.length; i++) {
	passport.serializeUser(function(user, done) {
		console.log("Serialize: ", user.email[i]);
		  done(null, user.id[i]);
	});
	
	passport.deserializeUser(function(id, done) {
	  	console.log("Deserialize: ", id);
	  	user = (userDB.id[i] === id) ? userDB : false;
	  	done(null, user);
	
	});
	
	passport.use(new LocalStrategy({	usernameField: 'email'	},
		function(email, password, done) {
		  	console.log(email + '/' + password)
		  	console.log(userDB.email[i].split(' ') + '/' + userDB.password[i].split(' '))
		  	if(password == userDB.password[i]) {
		  		console.log(userDB.email[i])
		  		return done(null,userDB)
		  	}
		  	else {
		  		console.log(userDB.password[i])
		  		return done(null,false)
		  	}
		})
	);
}

Answer 1

[Михаил]

Comments

[historydev]

Я выносил за функции ее и уже в запросе назначал ей массив,но она не меняется почему-то,я могу как-то это исправить?

[historydev]

Здесь создавал и внутри задавал значение,но результат сменился лишь на ошибку о том,что нельзя измерять длину null

[Михаил]

Владимир, научитесь пользоваться асинхронным кодом

[historydev]

Михаил, дайте источник где почитать пожалуйста,где есть решение подобной ситуации,спасибо

[Михаил]

Владимир,
https://blog.bitsrc.io/understanding-asynchronous-...

варианты:
1. Перенести цикл в ещё 1 .then
2. Перенести цикл в функцию
3. Использовать await на query

[historydev]

Михаил,2 пункт пробовал,а что писать в .then()?Какой параметр передать или этого можно не делать?Вот в примере await function f() {}
А как мне применить await в моем случае,ведь как я понял,нужно применять его на sequelize?

[Михаил]

Владимир,

sequalize.query
  .then((res1) => {
    return res2;
  })
  .then((res2) => {
    return res3;
  })
  .then((res3) => {
    return res4;
  });

или
(async () => {
  const data = await sequalize.query.then((...) => {
    return ...;
  });

  // вот тут будет data.userId
})()

[historydev]

Михаил, Куда я его только не пихал,везде ругается на него

[historydev]

Михаил, Дак у меня скрипт сравнивает кучу данных,после того как юзер кинет пост запрос,мне не нужно сразу обрабатывать цикл

[historydev]

DarthWazer, как переменную выносил,а не как константу,но спасибо

[Михаил]

Владимир,

sequelize
  .query('SELECT * FROM users', { raw: true })
  .then(users => {
    const isIds = users[0].map(user => user.id);
    const isEmails = users[0].map(user => user.email);
    const isPasswords = users[0].map(user => user.password);

    return {
      isIds: isIds,
      email: isEmails,
      password: isPasswords
    }
  })
  .then((userDB) => {
    for (i = 0; i < userDB.isIds.length; i++) {
      passport.serializeUser(function(user, done) {
        console.log("Serialize: ", user.email[i]);
        done(null, user.id[i]);
      });

      passport.deserializeUser(function(id, done) {
        console.log("Deserialize: ", id);
        user = (userDB.id[i] === id) ? userDB : false;
        done(null, user);
      });

      passport.use(new LocalStrategy({ usernameField: 'email' },
        function(email, password, done) {
          console.log(email + '/' + password)
          console.log(userDB.email[i].split(' ') + '/' + userDB.password[i].split(' '))
          if (password == userDB.password[i]) {
            console.log(userDB.email[i])
            return done(null, userDB)
          } else {
            console.log(userDB.password[i])
            return done(null, false)
          }
        }));
    }
  });

[historydev]

Михаил, если рассуждать с обработкой цикла сразу после запроса,то его достаточно засунуть в then

[Михаил]

Владимир, вообще это хрень какая-то, разнесите код по файлам и моделям

В итоге POST должен выглядеть примерно так, учитывая что passport нужно до этого подключать вообще-то

app.post('/', async (req, res) => {
  const user = await User.find(id);

  passport.authenticate(req);

  res.end('ok');
});

[historydev]

Михаил, У меня паспорт подключен до в app.js

[historydev]

Михаил, резалт вашего кода

TypeError: Cannot read property '40' of undefined
at C:\Users\History\Desktop\session\config.js:73:26
at pass (C:\Users\History\Desktop\session\node_modules\passport\lib\authenticator.js:357:9)
at Authenticator.deserializeUser (C:\Users\History\Desktop\session\node_modules\passport\lib\authenticator.js:362:5)
at SessionStrategy.authenticate (C:\Users\History\Desktop\session\node_modules\passport\lib\strategies\session.js:60:10)
at attempt (C:\Users\History\Desktop\session\node_modules\passport\lib\middleware\authenticate.js:361:16)
at authenticate (C:\Users\History\Desktop\session\node_modules\passport\lib\middleware\authenticate.js:362:7)
at Layer.handle [as handle_request] (C:\Users\History\Desktop\session\node_modules\express\lib\router\layer.js:95:5)
at trim_prefix (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:317:13)
at C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:284:7
at Function.process_params (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:335:12)
at next (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:275:10)
at initialize (C:\Users\History\Desktop\session\node_modules\passport\lib\middleware\initialize.js:53:5)
at Layer.handle [as handle_request] (C:\Users\History\Desktop\session\node_modules\express\lib\router\layer.js:95:5)
at trim_prefix (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:317:13)
at C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:284:7
at Function.process_params (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:335:12)
at next (C:\Users\History\Desktop\session\node_modules\express\lib\router\index.js:275:10)
at C:\Users\History\Desktop\session\node_modules\express-session\index.js:495:7
at readCallback (C:\Users\History\Desktop\session\node_modules\session-file-store\lib\session-file-helpers.js:184:20)
at C:\Users\History\Desktop\session\node_modules\graceful-fs\graceful-fs.js:115:16
at FSReqWrap.readFileAfterClose [as oncomplete] (internal/fs/read_file_context.js:53:3)

[Михаил]

Владимир,
C:\Users\History\Desktop\session\config.js:73:26

Мне ваш комп хакнуть чтоб код увидеть?

[historydev]

Михаил, извините,мой косяк,но данные undefined выдает..

[historydev]

app.js:

const express = require('express');
const session = require('express-session');
const FileStore = require('session-file-store')(session);
const passport = require('passport');
const app = express();
const port = 3000;

app.use(express.json());
app.use(express.urlencoded({extended: false}));

app.use(
	session({
		secret: "secret",
		store: new FileStore(),
		cookie: {
			path: "/",
			httpOnly: true,
			maxAge: 60 * 60 * 1000
		},
		resave: false,
		saveUnitialized: false
	})
);

require('./config');
app.use(passport.initialize());
app.use(passport.session());

const logout = (req,res,next) => {
	if(req.isAuthenticated()) {
		return res.redirect('/admin');
	} else {
		next()
	}
}

app.get('/',logout, (req, res) => res.sendFile(__dirname + '/index.html'));

app.post('/login', function(req, res, next) {
  passport.authenticate('local', function(err, user) {
    if (err) {
    	return next(err);
    }
    if (!user) { 
    	return res.redirect('/');
    	console.log(user);
    }
    req.logIn(user, function(err) {
      if (err) { 
      	return next(err);
      }
      return res.redirect('/admin');
    });
  })(req, res, next);
});

app.post('/register', function(req,res) {
	User.create({ email: "Jane", password: "Doe" }).then(jane => {
  		console.log("Jane's auto-generated ID:", jane.id);
	});
});

const auth = (req,res,next) => {
	if(req.isAuthenticated()) {
		next()
	} else {
		return res.redirect('/');
	}
}

app.get('/admin', auth, (req, res) => {
	res.send('Admin page');
});

app.get('/logout', (req,res) => {
	req.logout();
	res.redirect('/');
});

app.listen(port, () => console.log(`Example app listening on port ${port}!`));

config.js(Вернул свой код):

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

const Sequelize = require('sequelize');

// Option 1: Passing parameters separately
const sequelize = new Sequelize('test', 'root', '', {
  host: 'localhost',
  dialect: 'mysql',
  operatorsAliases: false,
  pool: {
    max: 5,
    min: 0,
    acquire: 30000,
    idle: 10000
  }

  // dialectOptions: {
  //   socketPath: '/Applications/MAMP/tmp/mysql/mysql.sock',
  //   supportBigNumbers: true,
  //   bigNumberStrings: true
  // },

});

sequelize
  .authenticate()
  .then(() => {
    console.log('Connection has been established successfully.');
  })
  .catch(err => {
    console.error('Unable to connect to the database:', err);
  });

const User = sequelize.define('users', {
  // attributes
  email: {
    type: Sequelize.STRING,
    allowNull: false
  },
  password: {
    type: Sequelize.STRING
    // allowNull defaults to true
  }
}, {
  // options
});

let userDB = null;

sequelize
  .query('SELECT * FROM users', { raw: true })
  .then(users => {
  	const isIds = users[0].map(user => user.id);
  	const isEmails = users[0].map(user => user.email);
  	const isPasswords = users[0].map(user => user.password);

  	userDB = {
		id: isIds,
		email: isEmails,
		password: isPasswords
	}
});

for(i=0; i<userDB.id.length; i++){
	passport.serializeUser(function(user, done) {
		console.log("Serialize: ", user.email[i]);
		  done(null, user.id[i]);
	});
	
	passport.deserializeUser(function(id, done) {
	  	console.log("Deserialize: ", id);
	  	user = (userDB.id[i] === id) ? userDB : false;
	  	done(null, user);
	
	});
	
	passport.use(new LocalStrategy({	usernameField: 'email'	},
		function(email, password, done) {
		  	console.log(email + '/' + password)
		  	console.log(userDB.email[i].split(' ') + '/' + userDB.password[i].split(' '))
		  	if(password == userDB.password[i]) {
		  		console.log(userDB.email[i])
		  		return done(null,userDB)
		  	}
		  	else {
		  		console.log(userDB.password[i])
		  		return done(null,false)
		  	}
		})
	);
}

[Михаил]

Владимир, разнесите код по отдельным функциям/промисам, каждая из которых будет выполнять 1 конкретную задачу

[Михаил]

Владимир, и что в этом коде нового? Та же ошибка с АСИНХРОННЫМ кодом, ваш for вызывается ДО получения данных с бд

[Михаил]

Владимир, и что вы сделать то пытаетесь вообще?

[historydev]

Михаил, Я стараюсь реализовать регистрацию и авторизацию с сессиями

[Михаил]

Владимир, доставая каждого пользователя из бд и записывая это в переменную?

[historydev]

Михаил, ну,а как мне еще проверять?Я просто так всегда делал..Есть способ валидный?

[Михаил]

Владимир, -_-
1. А если миллион пользователей будет в бд?
2. А если я сменю пароль и ваша password == userDB.password[i] уже не будет работать

Примеров использования пасспорта в инете полно, не знаю куда вы смотрели

https://medium.com/@johnnyszeto/node-js-user-authe...
https://www.sitepoint.com/local-authentication-usi...
https://riptutorial.com/node-js/example/27459/exam...

То, что вернёт passport.serializeUser будет записано в сессию, обычно user.id будет достаточно

passport.deserializeUser достаёт этот записанный id из сессии и дает вам, далее из модели User через .findOne достаёт пользователя из бд и записывает в req.user

[Михаил]

Владимир, и если есть req.user или req.isAuthenticated(), значит пользователь залогинен

[historydev]

Михаил, вы подтолкнули меня на дорогу,я нашел то,что искал в самом начале пути,вот здесь
То,что мне было нужно.Но теперь у меня новая проблема,у меня ответ пустой на запрос,почему-то..

passport.use(new LocalStrategy({	usernameField: 'email'	},
		function(email, password, done) {


		  	connection.query("SELECT * FROM users WHERE email LIKE '"+email+"'",	function(err,res){

				if (err) {
					return done(null,false);
					console.log(err);
				}

				else {

					console.log(res);
					return done(null,res);

				}

			});


				// if (rows.length) {
    //         	    return done(null, false, req.flash('signupMessage', 'That email is already taken.'));
    //         	}
    //         	else {

				// 	// if there is no user with that email
    //             	// create the user
    //             	var newUserMysql = new Object();
				
				// 	newUserMysql.email    = email;
    //             	newUserMysql.password = password; // use the generateHash function in our user model
			
				// 	var insertQuery = "INSERT INTO users ( email, password ) values ('" + email +"','"+ password +"')";
				// 	console.log(insertQuery);

				// 	connection.query(insertQuery,function(err,rows){
				// 		newUserMysql.id = rows.insertId;				
				// 		return done(null, newUserMysql);
				// 	});
				// }
		}
	))

[Михаил]

Владимир, откуда я знаю что у вас в бд и что в email

[Михаил]

Владимир,
1. Зачем вы email ищите по LIKE?
2. Как насчет [e-mail] и [e-mail] ?
3. "SELECT * FROM users WHERE email LIKE '"+email+"'" sql-injection что это да?

return done(null,false);
console.log(err);

после return функция не выполняется

[historydev]

Михаил, в любом случае получаю [] изначально было

"SELECT * FROM users WHERE email='"+email+"'"

[historydev]

Михаил, переставил вывод ошибки,ошибок в запросе нет,все чисто

[Михаил]

Владимир, запись в бд? значение переменной email?

[historydev]

Михаил, проверял,выводит то что я ввел,почту копировал из таблицы напрямую в поле,у меня было подобное когда копировал код,кодировка шалила,но здесь ошибок нет и я не могу понять в чем дело,там выскакивало мол юзаю не ту mysql версию

[Михаил]

Владимир,
SELECT * FROM users ?

[historydev]

Михаил, проходит

[Михаил]

Владимир, SELECT * FROM users WHERE email = '[e-mail]'

[historydev]

Михаил, такс,что-то не так,вбил запрос в pma,пусто,данные верно записаны

[historydev]

Михаил,

[Михаил]

Владимир,
SELECT * FROM test.users ?

[historydev]

Михаил, как я и предполагал,походу дело в кодировке,таблица была сгенерирована sequelize везде пусто в кодировке кроме email и пароля,там latin что-то там,сейчас пересоздам таблицу,посмотрим

[historydev]

Михаил, проблема решена, [ RowDataPacket { id: 1, email: '[e-mail]', password: 'Hancock' } ]

[historydev]

Михаил, спасибо,Михаил.

[Михаил]

Владимир, каким образом?

[historydev]

Михаил, удалил кодировку у полей