我正在使用带有express和mssql的节点,并且查询数据库工作正常。但是,如果我完全同时传递2个并发请求,则会得到:
UnhandledPromiseRejectionWarning:ConnectionError:数据库已连接!在连接到其他数据库之前,请先调用close。
var express = require('express'); // Web Framework
var app = express();
var sql = require('mssql/msnodesqlv8');
const pool = new sql.ConnectionPool({
database: 'db123',
server: 'server1',
driver: 'msnodesqlv8',
options: {
trustedConnection: true
},
})
var conn = pool;
var server = app.listen(5001, function () {
var host = server.address().address
var port = server.address().port
console.log("app listening at http://%s:%s", host, port)
});
// GET: SQL Stored procedure return time codes
app.get('/codes/:userid/:showclosed', function (req, res) {
conn.connect().then(function () {
var request = new sql.Request(conn);
request.input('userid', req.params.userid);
request.input('showclosed', parseInt(req.params.showclosed));
request.execute('sel_new', function(err, recordsets, returnValue, affected) {
if(err) console.log(err);
res.end(JSON.stringify(recordsets)); // Result in JSON format
conn.close();
})
})
})
编辑:
这就是我最后得到的:
app.get('/codes/:userid/:showclosed', function (req, res) {
//const pool1 = new sql.ConnectionPool(config);
(async function () {
try {
let pool = await conn;
// Stored procedure
let result2 = await pool.request()
.input('userid', req.params.userid)
.input('showclosed', parseInt(req.params.showclosed))
.execute('StoredProcedure1', function(err, recordsets, returnValue, affected) {
if(err) console.log(err);
res.end(JSON.stringify(recordsets)); // Result in JSON format
//sql.close();
})
} catch (err) {
console.log(err);
}
})()
sql.on('error', err => {
console.log(err);
})
})
答案 0 :(得分:0)
您不应在每个请求上都使用conn.connect()
,这将初始化连接池,并应在服务启动时使用。
因此,您希望conn.connect()
在var conn = pool
之后立即发生。这将指示连接池打开与数据库的连接,然后您的应用程序可以在各个端点中按需使用该数据库。您可以在mssql library documentation中看到一个示例。该代码打开与sql.connect()
的连接,然后继续使用sql
对象创建和执行SQL查询。在您的示例中,您将使用conn
而不是sql
。我没有亲自使用该库,但这就是文档所建议的。
在调用数据库后,您也不应该关闭连接,因为您可能没有想到这会关闭整个连接池。这个想法是打开一个连接池,并在服务运行期间一直保持连接状态,但是您应该在关闭阶段明确运行conn.close()
。