Бесконечная загрузка сайта, что делать?

Question

[Anonymous]

Здравствуйте, я делаю вход для пользователя. Регистрация уже сделана и работает успешно. Не могу написать вход, бесконечная загрузка. Я задавал похожий вопрос, но тут явно проблема в другом. Версия Node.js: V18. Используемые библиотеки: express.js, mysql2. Страница загружается успешно, но когда я отправляю форму - начинается бесконечная загрузка, но задумано, чтобы писало: "Вы вошли под именем {имя пользователя}". Ошибку не выводит в консоли NodeJS.

Код

app.post('/login', urlencodedParser, async(req, res) => {
    // Capture the input fields
    let username = req.body.username;
    let password = req.body.password;
    // Ensure the input fields exists and are not empty
    try {
        if (username && password) {
            // Execute SQL query that'll select the account from the database based on the specified username and password
            await dbConnection.query('SELECT * FROM users WHERE name = ? AND password = ?', [username, password], function(err, results, fields) {
                // If there is an issue with the query, output the error
                // If the account exists
                if (results.length > 0) {
                    // Authenticate the user
                    req.session.loggedin = true;
                    req.session.username = username;
                    // Redirect to home page
                    res.send(`Вы вошли под именем ${username}`);
                } else {
                    res.send('Неправильный пароль или имя пользователя');
                }
                res.end();

            });
        } else {
            res.send('Введите имя пользователя/пароль');
            res.end();
        }
    } catch (err) {
        throw err
    }
});

Таблица users, MySQL:

1. Имя: name. Тип: VARCHAR(255)
2. Имя: password. Тип: VARCHAR(255)
3. Имя: id. Тип: INT. Доп. параметры: PRIMARY, AUTO_INCREMENT

Comments

[imko]

Именно загрузка? Никакой скрипт его не вешает просто?

[Anonymous]

imko, нет, я тут увидел, вкладку закрыть забыл, и до сих пор 'загружается'

[Сергей Сергей]

https://qna.habr.com/q/1148520 здесь показано, как заставить

await dbConnection.query('SELECT * FROM users WHERE name = ? AND password = ?', [username, password], function(err, results, fields) {

работать, далее можно думать про ошибки, которые ещё вытянуть из колбэка надо.

[Anonymous]

Сергей Сергей, а как вытянуть из колбэка? Не понимаю. Честно, я уже запутался с колбэками и async/await

[Сергей Сергей]

Нужен resolve/reject, а для них нужен Promise. А будет Promise, будет await работать. Там ответ написан, объяснять дальше базовые вещи неинтересно.

[Anonymous]

Сергей Сергей, Что это такое?

[Anonymous]

Сергей Сергей, почему-то не работает. Ошибок в консоли нет

Код

await new Promise(async(resolve, reject) => {
            await dbConnection.query('SELECT * FROM users WHERE name = ? AND password = ?', [username, password], function(err, results, fields) {
                if (!err) {
                    // If there is an issue with the query, output the error
                    // If the account exists
                    resolve(results);
                    if (results.length > 0) {
                        // Authenticate the user
                        req.session.loggedin = true;
                        req.session.username = username;
                        // Redirect to home page
                        res.send(`Вы вошли под именем ${username}`);
                    } else {
                        res.send('Неправильный пароль или имя пользователя');
                    }
                    res.end();
                } else {
                    reject(err);
                }
            });
        });

[Сергей Сергей]

Нужно поколупаться в этих промисах ещё.

Это чистый дебаг сейчас происходить должен, нужно расставить console.log в разных местах, чтобы разобраться с потоком исполнения.

[Anonymous]

Сергей Сергей,
if(err) {
console.log(err)
}
или
if (err) throw err

[Сергей Сергей]

Все варианты хороши, нужно испытывать всё. Я бы и alert делал, если бы nodejs мог. Также важно указывать дополнительные параметры в console.log, чтобы видеть из какой точки выводятся записи.

console.log( 'before await' );
await new Promise(async(resolve, reject) => {
    console.log( 'before await 2' );
    await dbConnection.query('SELECT * FROM users WHERE name = ? AND password = ?', [username, password], function(err, results, fields) {
        console.log( 'begin callback' );
        if (!err) {
            console.log( 'no error branch', results );
            // If there is an issue with the query, output the error
            // If the account exists
            resolve(results);
            console.log( 'after resolve', results.length );
            if (results.length > 0) {
                // Authenticate the user
                req.session.loggedin = true;
                req.session.username = username;
                // Redirect to home page
                res.send(`Вы вошли под именем ${username}`);
            } else {
                res.send('Неправильный пароль или имя пользователя');
            }
            res.end();
        } else {
            console.log( 'has error branch', err );
            reject(err);
        }
    });
});
console.log( 'after await' );

[Anonymous]

Сергей Сергей, напечатало before await 2, и дальше не тащит
P.S. Ошибок нет

Answer 1

[Николай Савельев]

Первое

dbConnection.query('SELECT * FROM users WHERE name = ? AND password = ?', [username, password], function(err, results, fields)

Не боишься sql-injection? Так никто не делает уже давно.

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

Comments

[Anonymous]

Я думаю проблема в другом, а не в async/await.

[Anonymous]

как поставить обработчик? И что за такой обработчик? Уязвимость, через которую можно было сделать SQL-Injection исправлена. С БД все норм, соединение интернета даже не нужно, так как бд стоит на локалхосте на 80 порте.