Merge pull request #1543 from partik-mis/1542-fix

Fix for #1542.

Author: dhensby

CHANGELOG.txt

@@ -1,4 +1,20 @@
-v9.1.3 (2023-??-??)
+v9.3.1 (2023-09-05)
+-------------------
+[fix] Fix bug with msnodesqlv8 connection strings ((#1525)[https://github.com/tediousjs/node-mssql/pull/1525])
+
+v9.3.0 (2023-09-04)
+-------------------
+[new] Add AAD connection support to connection strings ((#1461)[https://github.com/tediousjs/node-mssql/pull/1461])
+
+v9.2.1 (2023-09-05)
+-------------------
+[fix] Fix bug with msnodesqlv8 connection strings ((#1525)[https://github.com/tediousjs/node-mssql/pull/1525])
+
+v9.2.0 (2023-08-28)
+-------------------
+[new] Use @tediousjs/connection-string library to build msnodesqlv8 connection strings ((#1525)[https://github.com/tediousjs/node-mssql/pull/1525])
+
+v9.1.3 (2023-08-08)
 -------------------
 [fix] Escape values that are added to the msnodesqlv8 connection string that we construct ((#1479)[https://github.com/tediousjs/node-mssql/pull/1479])
 

lib/msnodesqlv8/connection-pool.js

@@ -24,7 +24,7 @@ class ConnectionPool extends BaseConnectionPool {
       if (!this.config.connectionString) {
         cfg.conn_str = buildConnectionString({
           Driver: CONNECTION_DRIVER,
-          Server: this.config.options.instanceName ? `${this.config.server}\\${this.config.instanceName}` : `${this.config.server},${this.config.port}`,
+          Server: this.config.options.instanceName ? `${this.config.server}\\${this.config.options.instanceName}` : `${this.config.server},${this.config.port}`,
           Database: this.config.database,
           Uid: this.config.user,
           Pwd: this.config.password,