Узел mysql2 async-await try catch

Мой вопрос касается обработки ошибок в асинхронных запросах mysql в Nodejs. Я использую обертку обещаний mysql для создания пула соединений в nodejs.

const mysql = require('mysql2/promise');

const pool = mysql.createPool({
host: '192.168.0.1',
  user: 'root',
  database: 'h2biz_18_dev',
  waitForConnections: true,
  connectionLimit: 100,
  queueLimit: 0,
  password : 'abc'
})

Внутри блока захвата освобождение соединения не может быть вызвано, так как выдается ошибка, говорящая

Cannot read property 'release' of undefined

Нужно ли освобождать соединение в блоке catch, если при выполнении запроса выдается ошибка? если да, то как правильно выпускать соединение и перехватывать ошибки в асинхронном ожидании с mysql2?

router.post('/getHolidaysOfYear',async function (req, res, next) {      
    var conn;

try{
    conn = await pool.getConnection();
        // Do something with the connection
    var promise1 = conn.execute(`
    SELECT 
        CALENDAR.*
    FROM
        hrm_calendar_holiday_policy1 CALENDAR
    `);
    const values = await Promise.all([promise1]);
    conn.release();
    return res.status(200).send({success:true, data:values[0][0]});
}

catch(err){
    logger.error(API_NAME + 'error :' + err);
    conn.release();
    return res.status(500).send({success:false, message:'Query Error'});
}
});

1 ответ

Решение

Похоже, ошибка происходит еще до того, как conn инициализируется, так что внутри pool.getConnection();,

Поэтому в вашем catch-блоке переменная conn имеет значение null и поэтому не может / не должен быть освобожден.

Вы могли бы сделать что-то вроде:

catch(err){
    logger.error(API_NAME + 'error :' + err);
    if(conn !== undefined) //This will check if the conn has already be initialized
        conn.release();
    return res.status(500).send({success:false, message:'Query Error'});
}
Другие вопросы по тегам