why does this mysql error causes nodejs to crash instead of going to the catch function?

Question

I have a mysql statement that creates an entry, it has a .then function and a .catch function, but when the following error occurs:

TypeError('Bind parameters must not contain undefined. To pass SQL NULL specify JS null');

the server crashes instead of answering a 500 like defined in the .catch function

Note: I'm using the mysql2 library from npm with promises (require('mysql2/promise');)

Here's the code that calls it (req.params.account_name is undefined):

const CREATE_ACCOUNT_STATEMENT =
  'INSERT INTO `Accounts` (`account_token`, `account_name`) VALUES (?, ?)'

try {
  mysqlConnectionPool.execute(CREATE_ACCOUNT_STATEMENT, [
    account_token, account_name
  ])
  .then(() => {
    res.end(JSON.stringify({ token: account_token }))
  })
  .catch((e) => {
    debug(1, "error while trying to create account:", e)
    res.status(500).end("Internal Server Error")
  })
} catch(e) {
  debug(1, "error while trying to create account:", e)
  res.status(500).end("Internal Server Error")
}

Answer 1

Actually, @Quentine was close to the right thing...

It is "sort of" a bug in mysql2, i use sort-of because https://github.com/sidorares/node-mysql2/issues/902 suggests the development team of mysql2 is o.k. with it.

it is an issue with the way mysql2.pool passes the call to the created connection, which does not pass the exception to the wrapping promise.

I ended up making my own wrapping function to create the connection + call execute wrapped in proper promise handling.

import mysql = require('mysql2');
private async queryDB(query:string, useExecute: boolean = false, ...args:any[]) : Promise<any[]>
    {
        return new Promise<any[]>((resolve, reject)=>{
            for(var i = 0; i < args.length; ++i)
            {
                if(args[i]===undefined)
                    args[i] = null;
            }
            this.dbPool.getConnection((err, conn)=>{
                if(err){
                    reject(err);
                    return;
                }
                
                let cb = function(err: mysql.QueryError, results: any[], fields: mysql.FieldPacket[]) {
                    conn.release();
                    if(err)
                    {
                        reject(err);
                        return;
                    }
                    resolve(results);
                }
                if(useExecute)
                    conn.execute(query, args, cb);
                else
                    conn.query(query, args, cb);                
            });
        });
    }

Answer 2

mysqlConnectionPool.execute is throwing the exception before creating a promise.

i.e. the exception is not thrown from within the promise.

To catch it you would need to try {} catch (e) {} around the call to mysqlConnectionPool.execute.