Throw an error when a connection ends with a RST,ACK packet?

Hi,

I’m using Azure MySQL database.

According to MySQL docs, COM_QUIT command should end with “either a connection close or a OK_Packet”.

It can be reproduced when you run this example code against an Azure MySQL instance.

var mysql      = require('mysql2');
var connection = mysql.createConnection({ config });

connection.connect();

connection.query('SELECT 1 + 1 AS solution', function (error, results, fields) {
  if (error) throw error;
  console.log('The solution is: ', results[0].solution);
});

connection.end();

Throws ECONNRESET error.

Should this library throw an error when a connection end with a RST with the ACK flag set? Can we detect this?

I see mysqljs already fixed it at commit https://github.com/mysqljs/mysql/commit/9168ced54148721c8872ddab583116f00b4bc4f8 . But we prefer using mysql2 now and it block me. Could mysql2 fix it either?

[sidorares]

probably fix should be here - https://github.com/sidorares/node-mysql2/blob/5c5aa3aed7e772e599dbefdafb04405672901808/lib/connection.js#L123

if already _closing, ignore error ( same way .end() is expected when this flag set on line 94 )

[elemount]

I used the code in the _handleNetworkError and now it works. I raised a pull request on it.