mssql 11.0.0 → 12.0.0

package/README.md

@@ -1,16 +1,31 @@
 # node-mssql
 
-Microsoft SQL Server client for Node.js
+**Microsoft SQL Server client for Node.js**
 
 [![NPM Version][npm-image]][npm-url] [![NPM Downloads][downloads-image]][downloads-url] [![Appveyor CI][appveyor-image]][appveyor-url] [![Join the chat at https://gitter.im/patriksimek/node-mssql](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/patriksimek/node-mssql?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
 Supported TDS drivers:
+
 - [Tedious][tedious-url] (pure JavaScript - Windows/macOS/Linux, default)
-- [Microsoft / Contributors Node V8 Driver for Node.js for SQL Server][msnodesqlv8-url] (v2 native - Windows or Linux/macOS 64 bits only)
+- [MSNodeSQLv8][msnodesqlv8-url] (Microsoft / Contributors Node V8 Driver for Node.js for SQL Server, v2 native - Windows or Linux/macOS 64 bits only)
 
 ## Installation
 
-    npm install mssql
+### Tedious driver (default)
+
+```
+npm install mssql
+```
+
+### MSNodeSQLv8 driver (optional)
+
+```
+npm install mssql msnodesqlv8
+```
+
+## SQL Server prerequisites
+
+This package requires TCP/IP to connect to SQL Server, and you may need to enable this in your installation.
 
 ## Short Example: Use Connect String
 
@@ -68,6 +83,32 @@ const sqlConfig = {
 })()
 ```
 
+## Windows Authentication Example Using MSNodeSQLv8
+
+```javascript
+const sql = require('mssql/msnodesqlv8');
+
+const config = {
+  server: "MyServer",
+  database: "MyDatabase",
+  options: {
+    trustedConnection: true, // Set to true if using Windows Authentication
+    trustServerCertificate: true, // Set to true if using self-signed certificates
+  },
+  driver: "msnodesqlv8", // Required if using Windows Authentication
+};
+
+(async () => {
+  try {
+    await sql.connect(config);
+    const result = await sql.query`select TOP 10 * from MyTable`;
+    console.dir(result);
+  } catch (err) {
+    console.error(err);
+  }
+})();
+```
+
 ## Documentation
 
 ### Examples
@@ -87,7 +128,7 @@ const sqlConfig = {
 ### Drivers
 
 * [Tedious](#tedious)
-* [Microsoft / Contributors Node V8 Driver for Node.js for SQL Server](#microsoft--contributors-node-v8-driver-for-nodejs-for-sql-server)
+* [MSNodeSQLv8](#msnodesqlv8)
 
 ### Connections
 
@@ -174,16 +215,16 @@ const sql = require('mssql')
         let result1 = await pool.request()
             .input('input_parameter', sql.Int, value)
             .query('select * from mytable where id = @input_parameter')
-            
+
         console.dir(result1)
-    
+
         // Stored procedure
-        
+
         let result2 = await pool.request()
             .input('input_parameter', sql.Int, value)
             .output('output_parameter', sql.VarChar(50))
             .execute('procedure_name')
-        
+
         console.dir(result2)
     } catch (err) {
         // ... error checks
@@ -208,7 +249,7 @@ sql.on('error', err => {
 
 sql.connect(config).then(pool => {
     // Query
-    
+
     return pool.request()
         .input('input_parameter', sql.Int, value)
         .query('select * from mytable where id = @input_parameter')
@@ -229,9 +270,9 @@ sql.on('error', err => {
 })
 
 sql.connect(config).then(pool => {
-    
+
     // Stored procedure
-    
+
     return pool.request()
         .input('input_parameter', sql.Int, value)
         .output('output_parameter', sql.VarChar(50))
@@ -263,7 +304,7 @@ sql.on('error', err => {
 })
 ```
 
-All values are automatically sanitized against sql injection. 
+All values are automatically sanitized against sql injection.
 This is because it is rendered as prepared statement, and thus all limitations imposed in MS SQL on parameters apply.
 e.g. Column names cannot be passed/set in statements using variables.
 
@@ -310,7 +351,7 @@ sql.on('error', err => {
 
 ### Streaming
 
-If you plan to work with large amount of rows, you should always use streaming. Once you enable this, you must listen for events to receive data.
+If you plan to work with large amount of rows, you should always use streaming. Once you enable this, you must listen for events to receive data. Events must be attached before the query completes, but can be attached while in-flight.
 
 ```javascript
 const sql = require('mssql')
@@ -320,7 +361,6 @@ sql.connect(config, err => {
 
     const request = new sql.Request()
     request.stream = true // You can set streaming differently for each request
-    request.query('select * from verylargetable') // or request.execute(procedure)
 
     request.on('recordset', columns => {
         // Emitted once for each recordset in a query
@@ -342,6 +382,8 @@ sql.connect(config, err => {
     request.on('done', result => {
         // Always emitted as the last one
     })
+    
+    request.query('select * from verylargetable') // or request.execute(procedure)
 })
 
 sql.on('error', err => {
@@ -580,7 +622,7 @@ In addition to configuration object there is an option to pass config as a conne
 ```
 Server=localhost,1433;Database=database;User Id=username;Password=password;Encrypt=true
 ```
-###### Standard configuration using msnodesqlv8 driver
+###### Standard configuration using MSNodeSQLv8 driver
 ```
 Driver=msnodesqlv8;Server=(local)\INSTANCE;Database=database;UID=DOMAIN\username;PWD=password;Encrypt=true
 ```
@@ -648,12 +690,21 @@ On top of the extra options, an `authentication` property can be added to the po
 - **authentication** - An object with authentication settings, according to the [Tedious Documentation](https://tediousjs.github.io/tedious/api-connection.html). Passing this object will override `user`, `password`, `domain` settings.
 - **authentication.type** - Type of the authentication method, valid types are `default`, `ntlm`, `azure-active-directory-password`, `azure-active-directory-access-token`, `azure-active-directory-msi-vm`, or `azure-active-directory-msi-app-service`
 - **authentication.options** - Options of the authentication required by the `tedious` driver, depends on `authentication.type`. For more details, check [Tedious Authentication Interfaces](https://github.com/tediousjs/tedious/blob/v11.1.1/src/connection.ts#L200-L318)
+- `tedious` does not support Windows Authentication/Trusted Connection, however the `msnodesqlv8` driver does.
 
 More information about Tedious specific options: http://tediousjs.github.io/tedious/api-connection.html
 
-### Microsoft / Contributors Node V8 Driver for Node.js for SQL Server
+___
+
+### MSNodeSQLv8
 
-**Requires Node.js v10+ or newer. Windows 32-64 bits or Linux/macOS 64 bits only.** This driver is not part of the default package and must be installed separately by `npm install msnodesqlv8@^2`. To use this driver, use this require syntax: `const sql = require('mssql/msnodesqlv8')`.
+Alternative driver, requires Node.js v10+ or newer; Windows (32 or 64-bit) or Linux/macOS (64-bit only). It's not part of the default package so it must be [installed](#msnodesqlv8-driver) in addition. Supports [Windows/Trusted Connection authentication](#windows-authentication-example-using-msnodesqlv8).
+
+**To use this driver you must use this `require` statement:**
+
+```javascript
+const sql = require('mssql/msnodesqlv8')
+```
 
 Note: If you use import into your lib to prepare your request (`const { VarChar } = require('mssql')`) you also need to upgrade all your types import into your code (`const { VarChar } = require('mssql/msnodesqlv8')`) or a `connection.on is not a function` error will be thrown.
 
@@ -678,6 +729,8 @@ Driver={SQL Server Native Client 11.0};Server={#{server}\\#{instance}};Database=
 
 Please note that the connection string with this driver is not the same than tedious and use yes/no instead of true/false. You can see more on the [ODBC](https://docs.microsoft.com/fr-fr/dotnet/api/system.data.odbc.odbcconnection.connectionstring?view=dotnet-plat-ext-5.0) documentation.
 
+___
+
 ## Connections
 
 Internally, each `ConnectionPool` instance is a separate pool of TDS connections. Once you create a new `Request`/`Transaction`/`Prepared Statement`, a new TDS connection is acquired from the pool and reserved for desired action. Once the action is complete, connection is released back to the pool. Connection health check is built-in so once the dead connection is discovered, it is immediately replaced with a new one.
@@ -1341,7 +1394,7 @@ ps.prepare('select @param as value', err => {
 
         console.log(result.recordset[0].value) // return 12345
         console.log(result.rowsAffected) // Returns number of affected rows in case of INSERT, UPDATE or DELETE statement.
-        
+
         ps.unprepare(err => {
             // ... error checks
         })
@@ -1374,9 +1427,9 @@ ps.prepare('select @param as value', err => {
 
     request.on('done', result => {
         // Always emitted as the last one
-        
+
         console.log(result.rowsAffected) // Returns number of affected rows in case of INSERT, UPDATE or DELETE statement.
-        
+
         ps.unprepare(err => {
             // ... error checks
         })
@@ -1682,7 +1735,7 @@ More information about JSON support can be found in [official documentation](htt
 
 ## Handling Duplicate Column Names
 
-If your queries contain output columns with identical names, the default behaviour of `mssql` will only return column metadata for the last column with that name. You will also not always be able to re-assemble the order of output columns requested. 
+If your queries contain output columns with identical names, the default behaviour of `mssql` will only return column metadata for the last column with that name. You will also not always be able to re-assemble the order of output columns requested.
 
 Default behaviour:
 ```javascript
@@ -2067,6 +2120,16 @@ request.query('select @myval as myval', (err, result) => {
 - If you're facing problems with connecting SQL Server 2000, try setting the default TDS version to 7.1 with `config.options.tdsVersion = '7_1'` ([issue](https://github.com/tediousjs/node-mssql/issues/36))
 - If you're executing a statement longer than 4000 chars on SQL Server 2000, always use [batch](#batch-batch-callback) instead of [query](#query-command-callback) ([issue](https://github.com/tediousjs/node-mssql/issues/68))
 
+## 10.x to 11.x changes
+
+- Upgraded to tedious version 18
+- Dropped support for Node version <=16
+
+## 9.x to 10.x changes
+
+- Upgraded to tedious version 16
+- Dropped support for Node version <= 14
+
 ## 8.x to 9.x changes
 
 - Upgraded to tedious version 15

package/index.js

@@ -1 +1,3 @@
+'use strict'
+
 module.exports = require('./lib/tedious')

package/lib/base/connection-pool.js

@@ -7,7 +7,6 @@ const tarn = require('tarn')
 const { IDS } = require('../utils')
 const ConnectionError = require('../error/connection-error')
 const shared = require('../shared')
-const clone = require('rfdc/default')
 const { MSSQLError } = require('../error')
 
 /**
@@ -53,7 +52,7 @@ class ConnectionPool extends EventEmitter {
         throw ex
       }
     } else {
-      this.config = clone(config)
+      this.config = config
     }
 
     // set defaults

package/lib/base/request.js

@@ -53,6 +53,7 @@ class Request extends EventEmitter {
    * @param {Template literal} template
    * @return {String}
    */
+
   template () {
     const values = Array.prototype.slice.call(arguments)
     const strings = values.shift()
@@ -101,7 +102,7 @@ class Request extends EventEmitter {
    * Add an input parameter to the request.
    *
    * @param {String} name Name of the input parameter without @ char.
-   * @param {*} [type] SQL data type of input parameter. If you omit type, module automaticaly decide which SQL data type should be used based on JS data type.
+   * @param {*} [type] SQL data type of input parameter. If you omit type, module automatically decides which SQL data type should be used based on JS data type.
    * @param {*} value Input parameter value. `undefined` and `NaN` values are automatically converted to `null` values.
    * @return {Request}
    */
@@ -147,7 +148,7 @@ class Request extends EventEmitter {
    * Replace an input parameter on the request.
    *
    * @param {String} name Name of the input parameter without @ char.
-   * @param {*} [type] SQL data type of input parameter. If you omit type, module automaticaly decide which SQL data type should be used based on JS data type.
+   * @param {*} [type] SQL data type of input parameter. If you omit type, module automatically decides which SQL data type should be used based on JS data type.
    * @param {*} value Input parameter value. `undefined` and `NaN` values are automatically converted to `null` values.
    * @return {Request}
    */
@@ -253,7 +254,7 @@ class Request extends EventEmitter {
       return this
     }
 
-    // Check is method was called as tagged template
+    // Check if method was called as tagged template
     if (typeof batch === 'object') {
       const values = Array.prototype.slice.call(arguments)
       const strings = values.shift()
@@ -376,6 +377,7 @@ class Request extends EventEmitter {
    * @param {Object} streamOptions - optional options to configure the readable stream with like highWaterMark
    * @return {Stream}
    */
+
   toReadableStream (streamOptions = {}) {
     this.stream = true
     this.pause()
@@ -407,6 +409,7 @@ class Request extends EventEmitter {
    * @param {Stream} stream Stream to pipe data into.
    * @return {Stream}
    */
+
   pipe (writableStream) {
     const readableStream = this.toReadableStream()
     return readableStream.pipe(writableStream)
@@ -450,7 +453,7 @@ class Request extends EventEmitter {
       return this
     }
 
-    // Check is method was called as tagged template
+    // Check if method was called as tagged template
     if (typeof command === 'object') {
       const values = Array.prototype.slice.call(arguments)
       const strings = values.shift()

package/lib/datatypes.js

@@ -1,4 +1,5 @@
 'use strict'
+
 const objectHasProperty = require('./utils').objectHasProperty
 const inspect = Symbol.for('nodejs.util.inspect.custom')
 

package/lib/shared.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const TYPES = require('./datatypes').TYPES
+const { TYPES } = require('./datatypes')
 const Table = require('./table')
 
 let PromiseLibrary = Promise
@@ -57,7 +57,6 @@ const getTypeByValue = function (value) {
       return TYPES.NVarChar
 
     case 'number':
-    case 'bigint':
       if (value % 1 === 0) {
         if (value < -2147483648 || value > 2147483647) {
           return TYPES.BigInt
@@ -68,6 +67,13 @@ const getTypeByValue = function (value) {
         return TYPES.Float
       }
 
+    case 'bigint':
+      if (value < -2147483648n || value > 2147483647n) {
+        return TYPES.BigInt
+      } else {
+        return TYPES.Int
+      }
+
     case 'boolean':
       for (const item of Array.from(map)) {
         if (item.js === Boolean) {

package/lib/utils.js

@@ -1,3 +1,5 @@
+'use strict'
+
 const IDS = new WeakMap()
 const INCREMENT = {
   Connection: 1,

package/msnodesqlv8.js

@@ -1 +1,3 @@
+'use strict'
+
 module.exports = require('./lib/msnodesqlv8')

package/package.json

@@ -21,27 +21,33 @@
     "azure",
     "node-mssql"
   ],
-  "version": "11.0.0",
+  "version": "12.0.0",
   "main": "index.js",
   "type": "commonjs",
-  "repository": "github:tediousjs/node-mssql",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/tediousjs/node-mssql.git"
+  },
+  "homepage": "https://github.com/tediousjs/node-mssql#readme",
+  "bugs": {
+    "url": "https://github.com/tediousjs/node-mssql/issues"
+  },
   "license": "MIT",
   "dependencies": {
-    "@tediousjs/connection-string": "^0.5.0",
+    "@tediousjs/connection-string": "^0.6.0",
     "commander": "^11.0.0",
     "debug": "^4.3.3",
-    "rfdc": "^1.3.0",
     "tarn": "^3.0.2",
-    "tedious": "^18.2.1"
+    "tedious": "^19.0.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "^19.3.0",
-    "@commitlint/config-conventional": "^19.2.2",
+    "@commitlint/cli": "^20.0.0",
+    "@commitlint/config-conventional": "^20.0.0",
     "@semantic-release/commit-analyzer": "^11.1.0",
     "@semantic-release/github": "^9.2.6",
     "@semantic-release/npm": "^11.0.3",
     "@semantic-release/release-notes-generator": "^12.1.0",
-    "mocha": "^10.0.0",
+    "mocha": "^11.0.1",
     "semantic-release": "^22.0.12",
     "standard": "^17.0.0"
   },
@@ -64,6 +70,6 @@
     "test-cli": "mocha --exit -t 15000 test/common/cli.js"
   },
   "bin": {
-    "mssql": "./bin/mssql"
+    "mssql": "bin/mssql"
   }
 }

package/tedious.js

@@ -1 +1,3 @@
+'use strict'
+
 module.exports = require('./lib/tedious')