mssql 8.1.0 → 8.1.3

package/CHANGELOG.txt

@@ -1,3 +1,16 @@
+v8.1.3 (2022-08-08)
+-------------------
+[fix] requestTimeout correctly resolved ([#1398](https://github.com/tediousjs/node-mssql/pull/1398))
+[fix] Forcibly deny use of `useColumnNames` tedious config option that can be passed in the config object ([#1416](https://github.com/tediousjs/node-mssql/pull/1416))
+
+v8.1.2 (2022-05-27)
+-------------------
+[fix] quote identifiers in table constraint declaration ([#1397](https://github.com/tediousjs/node-mssql/pull/1397))
+
+v8.1.1 (2022-05-18)
+-------------------
+[fix] quote identifiers in table primary keys ([#1394](https://github.com/tediousjs/node-mssql/pull/1394))
+
 v8.1.0 (2022-04-06)
 -------------------
 [new] MSSQL CLI tool now accepts some options to allow overriding config file ((#1381](https://github.com/tediousjs/node-mssql/pull/1381))
@@ -26,6 +39,23 @@ v8.0.0 (2022-01-30)
 [removed] Remove connection string parser helpers ((#1342)[https://github.com/tediousjs/node-mssql/pull/1342])
 [removed] Remove deprecated pool properties ((#1359)[https://github.com/tediousjs/node-mssql/pull/1359])
 
+v7.3.5 (2022-05-27)
+-------------------
+[fix] quote identifiers in table constraint declaration ([#1397](https://github.com/tediousjs/node-mssql/pull/1397))
+
+v7.3.4 (2022-05-18)
+-------------------
+[fix] msnodesqlv8.js file was missing from previous release
+
+v7.3.3 (2022-05-18)
+-------------------
+Duplicate tag
+
+v7.3.2 (2022-05-18)
+-------------------
+[fix] nodemsqlv8 driver tests working against Node 10 ([#1368](https://github.com/tediousjs/node-mssql/pull/1368))
+[fix] quote identifiers in table primary keys ([#1394](https://github.com/tediousjs/node-mssql/pull/1394))
+
 v7.3.1 (2022-02-24)
 -------------------
 [fix] Bump debug dependency ([#1361](https://github.com/tediousjs/node-mssql/pull/1361))

package/README.md

@@ -562,7 +562,7 @@ const config = {
 - **pool.max** - The maximum number of connections there can be in the pool (default: `10`).
 - **pool.min** - The minimum of connections there can be in the pool (default: `0`).
 - **pool.idleTimeoutMillis** - The Number of milliseconds before closing an unused connection (default: `30000`).
-- **arrayRowMode** - Return row results as a an array instead of a keyed object. Also adds `columns` array. See [Handling Duplicate Column Names](#handling-duplicate-column-names)
+- **arrayRowMode** - Return row results as a an array instead of a keyed object. Also adds `columns` array. (default: `false`) See [Handling Duplicate Column Names](#handling-duplicate-column-names)
 
 Complete list of pool options can be found [here](https://github.com/vincit/tarn.js/#usage).
 

package/lib/base/connection-pool.js

@@ -8,6 +8,7 @@ const { IDS } = require('../utils')
 const ConnectionError = require('../error/connection-error')
 const shared = require('../shared')
 const clone = require('rfdc/default')
+const { MSSQLError } = require('../error')
 
 /**
  * Class ConnectionPool.
@@ -68,6 +69,14 @@ class ConnectionPool extends EventEmitter {
       this.config.options.instanceName = RegExp.$2
     }
 
+    if (typeof this.config.options.useColumnNames !== 'undefined' && this.config.options.useColumnNames !== true) {
+      const ex = new MSSQLError('Invalid options `useColumnNames`, use `arrayRowMode` instead')
+      if (typeof callback === 'function') {
+        return setImmediate(callback, ex)
+      }
+      throw ex
+    }
+
     if (typeof callback === 'function') {
       this.connect(callback)
     }

package/lib/base/request.js

@@ -48,7 +48,7 @@ class Request extends EventEmitter {
   }
 
   /**
-   * Generate sql string and set imput parameters from tagged template string.
+   * Generate sql string and set input parameters from tagged template string.
    *
    * @param {Template literal} template
    * @return {String}

package/lib/msnodesqlv8/connection-pool.js

@@ -21,13 +21,11 @@ class ConnectionPool extends BaseConnectionPool {
         defaultConnectionString = CONNECTION_STRING_NAMED_INSTANCE
       }
 
-      if (this.config.requestTimeout == null) {
-        this.config.requestTimeout = 15000
-      }
+      this.config.requestTimeout = this.config.requestTimeout || this.config.timeout || 15000
 
       const cfg = {
         conn_str: this.config.connectionString || defaultConnectionString,
-        conn_timeout: (this.config.connectionTimeout || 15000) / 1000
+        conn_timeout: (this.config.connectionTimeout || this.config.timeout || 15000) / 1000
       }
 
       cfg.conn_str = cfg.conn_str.replace(/#{([^}]*)}/g, (p) => {

package/lib/msnodesqlv8/request.js

@@ -369,7 +369,7 @@ class Request extends BaseRequest {
           }
           const param = this.parameters[name]
           if (param.io === 1 || (param.io === 2 && param.value)) {
-            params.push(castParameter(param.value, param.type, param))
+            params.push(castParameter(param.value, param.type))
           }
         }
 

package/lib/table.js

@@ -99,7 +99,7 @@ Table.prototype._makeBulk = function _makeBulk () {
 }
 
 Table.prototype.declare = function declare () {
-  const pkey = this.columns.filter(col => col.primary === true).map(col => col.name)
+  const pkey = this.columns.filter(col => col.primary === true).map(col => `[${col.name}]`)
   const cols = this.columns.map(col => {
     const def = [`[${col.name}] ${declareType(col.type, col)}`]
 
@@ -116,7 +116,7 @@ Table.prototype.declare = function declare () {
     return def.join(' ')
   })
 
-  const constraint = pkey.length > 1 ? `, constraint PK_${this.temporary ? this.name.substr(1) : this.name} primary key (${pkey.join(', ')})` : ''
+  const constraint = pkey.length > 1 ? `, constraint [PK_${this.temporary ? this.name.substr(1) : this.name}] primary key (${pkey.join(', ')})` : ''
   return `create table ${this.path} (${cols.join(', ')}${constraint})`
 }
 

package/lib/tedious/connection-pool.js

@@ -37,7 +37,7 @@ class ConnectionPool extends BaseConnectionPool {
       cfg.options.database = cfg.options.database || this.config.database
       cfg.options.port = cfg.options.port || this.config.port
       cfg.options.connectTimeout = cfg.options.connectTimeout || this.config.connectionTimeout || this.config.timeout || 15000
-      cfg.options.requestTimeout = cfg.options.requestTimeout || this.config.requestTimeout != null ? this.config.requestTimeout : 15000
+      cfg.options.requestTimeout = cfg.options.requestTimeout || this.config.requestTimeout || this.config.timeout || 15000
       cfg.options.tdsVersion = cfg.options.tdsVersion || '7_4'
       cfg.options.rowCollectionOnDone = cfg.options.rowCollectionOnDone || false
       cfg.options.rowCollectionOnRequestCompletion = cfg.options.rowCollectionOnRequestCompletion || false

package/package.json

@@ -21,7 +21,7 @@
     "azure",
     "node-mssql"
   ],
-  "version": "8.1.0",
+  "version": "8.1.3",
   "main": "index.js",
   "repository": "github:tediousjs/node-mssql",
   "license": "MIT",