Правильно ли я делаю подключение и запрос к БД?

Question

[Redrik_Shuhart]

Изучаю node и пытаюсь сделать пет-проект c rest api. Правильно ли я делаю или есть ошибки? Надо закрывать соединения или нет?

db.js

const mysql = require('mysql2');

const pool = mysql.createPool({
  host: 'XXX',
  user: 'XXX',
  password: 'XXX',
  database: 'XXX',
  waitForConnections: true,
  connectionLimit: 10,
  queueLimit: 0,
  charset : 'UTF8MB4_UNICODE_CI',
  multipleStatements: true
});

export {pool}

api.js

import {pool} from "db";
const promisePool = pool.promise()
const express = require('express');
const app = express();

app.get('/api', async function (req, res) {

	const sql = `SELECT * FROM ........`

	try{
		const[rows] = await promisePool.query(sql)
		res.json(rows)
	}
	catch (e) {
		console.log(e)
	}

});

Answer 1

[Афанасий Захаров]

Я не эксперт, но, кажется, const sql = `SELECT * FROM ........`

Плохая идея, так как создает уязвимость к sql-инъекциям

Comments

[Redrik_Shuhart]

`SELECT * FROM ........` я от балды сейчас написал, при запросах я использую подготовленные запросы

Меня больше интересует правильно ли всё остальное: использование async, pool?