refactor!: remove deprecated items

Removals are noted in JIRA ticket.

NODE-2317

Author: nbbeeken

.eslintrc.json

@@ -7,7 +7,6 @@
   "plugins": [
     "@typescript-eslint",
     "prettier",
-    "promise",
     "eslint-plugin-tsdoc"
   ],
   "extends": [
@@ -27,10 +26,9 @@
 
     "tsdoc/syntax": "warn",
 
-    "no-console": "off",
+    "no-console": "error",
     "eqeqeq": ["error", "always", { "null": "ignore" }],
     "strict": ["error", "global"],
-    "promise/no-native": "error",
 
     "@typescript-eslint/no-explicit-any": "off"
   },

package-lock.json

@@ -31,8 +31,8 @@
   },
   "devDependencies": {
     "@istanbuljs/nyc-config-typescript": "^1.0.1",
-    "@microsoft/api-extractor": "^7.12.1",
-    "@microsoft/tsdoc-config": "^0.13.9",
+    "@microsoft/api-extractor": "^7.13.1",
+    "@microsoft/tsdoc-config": "^0.14.0",
     "@types/aws4": "^1.5.1",
     "@types/bl": "^2.1.0",
     "@types/chai": "^4.2.14",
@@ -42,19 +42,18 @@
     "@types/node": "^14.6.4",
     "@types/saslprep": "^1.0.0",
     "@types/semver": "^7.3.4",
-    "@typescript-eslint/eslint-plugin": "^3.10.0",
-    "@typescript-eslint/parser": "^3.10.0",
+    "@typescript-eslint/eslint-plugin": "^4.15.1",
+    "@typescript-eslint/parser": "^4.15.1",
     "chai": "^4.2.0",
     "chai-subset": "^1.6.0",
     "chalk": "^4.1.0",
     "co": "4.6.0",
     "coveralls": "^3.0.11",
-    "eslint": "^6.8.0",
+    "eslint": "^7.20.0",
     "eslint-config-prettier": "^6.11.0",
     "eslint-plugin-jsdoc": "^30.7.8",
     "eslint-plugin-prettier": "^3.1.3",
-    "eslint-plugin-promise": "^4.2.1",
-    "eslint-plugin-tsdoc": "^0.2.10",
+    "eslint-plugin-tsdoc": "^0.2.11",
     "js-yaml": "^3.14.0",
     "jsdoc": "^3.6.4",
     "lodash.camelcase": "^4.3.0",
@@ -72,9 +71,9 @@
     "source-map-support": "^0.5.19",
     "standard-version": "^8.0.2",
     "through2": "^3.0.1",
-    "ts-node": "^9.0.0",
-    "typedoc": "^0.20.14",
-    "typescript": "^4.0.5",
+    "ts-node": "^9.1.1",
+    "typedoc": "^0.20.25",
+    "typescript": "^4.1.5",
     "typescript-cached-transpile": "^0.0.6",
     "worker-farm": "^1.5.0",
     "wtfnode": "^0.8.2",

package.json

@@ -2,7 +2,7 @@ import * as crypto from 'crypto';
 import { Binary, Document } from '../../bson';
 import { MongoError, AnyError } from '../../error';
 import { AuthProvider, AuthContext } from './auth_provider';
-import { Callback, ns } from '../../utils';
+import { Callback, emitWarning, ns } from '../../utils';
 import type { MongoCredentials } from './mongo_credentials';
 import type { HandshakeDocument } from '../connect';
 
@@ -25,7 +25,7 @@ class ScramSHA extends AuthProvider {
       return callback(new MongoError('AuthContext must provide credentials.'));
     }
     if (cryptoMethod === 'sha256' && saslprep == null) {
-      console.warn('Warning: no saslprep library specified. Passwords will not be sanitized');
+      emitWarning('Warning: no saslprep library specified. Passwords will not be sanitized');
     }
 
     crypto.randomBytes(24, (err, nonce) => {

src/cmap/auth/scram.ts

@@ -5,6 +5,7 @@ import { OP_QUERY, OP_GETMORE, OP_KILL_CURSORS, OP_MSG } from './wire_protocol/c
 import type { Long, Document, BSONSerializeOptions } from '../bson';
 import type { ClientSession } from '../sessions';
 import type { CommandOptions } from './connection';
+import { MongoError } from '../error';
 
 // Incrementing request id
 let _requestId = 0;
@@ -826,7 +827,7 @@ export class BinMsg {
     while (this.index < this.data.length) {
       const payloadType = this.data.readUInt8(this.index++);
       if (payloadType === 1) {
-        console.error('TYPE 1');
+        throw new MongoError('Type 1 payload'); // ???
       } else if (payloadType === 0) {
         const bsonSize = this.data.readUInt32LE(this.index);
         const bin = this.data.slice(this.index, this.index + bsonSize);

src/cmap/commands.ts

@@ -1,7 +1,7 @@
 import Denque = require('denque');
 import { EventEmitter } from 'events';
 import { Logger } from '../logger';
-import { Connection, ConnectionOptions, CommandOptions } from './connection';
+import { Connection, ConnectionOptions } from './connection';
 import { connect } from './connect';
 import { eachAsync, relayEvents, makeCounter, Callback } from '../utils';
 import { MongoError } from '../error';
@@ -18,7 +18,6 @@ import {
   ConnectionCheckedInEvent,
   ConnectionPoolClearedEvent
 } from './events';
-import type { Document } from '../bson';
 
 const kLogger = Symbol('logger');
 const kConnections = Symbol('connections');
@@ -360,33 +359,6 @@ export class ConnectionPool extends EventEmitter {
       });
     });
   }
-
-  // NOTE: remove `isConnected` and `write` as part of NODE-2745
-  // These functions only exist if makeServerTrampoline is
-  // called when using the wire protocol methods
-
-  /**
-   * @internal
-   * @deprecated Remove sever trampoline code. (NODE-2745)
-   */
-  isConnected(): boolean {
-    throw new TypeError('This is not a server trampoline instance');
-  }
-
-  /**
-   * @internal
-   * @deprecated Remove sever trampoline code. (NODE-2745)
-   */
-  write(
-    message: Document,
-    commandOptions: CommandOptions,
-    callback: (err: MongoError, ...args: Document[]) => void
-  ): void {
-    message;
-    commandOptions;
-    callback;
-    throw new TypeError('This is not a server trampoline instance');
-  }
 }
 
 function ensureMinPoolSize(pool: ConnectionPool) {

src/cmap/connection_pool.ts

@@ -1,6 +1,6 @@
 import { GetMore, KillCursor, Msg, WriteProtocolMessageType } from './commands';
 import { calculateDurationInMs, deepCopy } from '../utils';
-import type { ConnectionPool, ConnectionPoolOptions } from './connection_pool';
+import { ConnectionPool, ConnectionPoolOptions } from './connection_pool';
 import type { Connection } from './connection';
 import type { Document } from '../bson';
 import type { AnyError } from '../error';
@@ -150,17 +150,17 @@ export class ConnectionPoolClearedEvent extends ConnectionPoolMonitoringEvent {
 }
 
 export const CMAP_EVENT_NAMES = [
-  'connectionPoolCreated',
-  'connectionPoolClosed',
-  'connectionCreated',
-  'connectionReady',
-  'connectionClosed',
-  'connectionCheckOutStarted',
-  'connectionCheckOutFailed',
-  'connectionCheckedOut',
-  'connectionCheckedIn',
-  'connectionPoolCleared'
-];
+  ConnectionPool.CONNECTION_POOL_CREATED,
+  ConnectionPool.CONNECTION_POOL_CLOSED,
+  ConnectionPool.CONNECTION_CREATED,
+  ConnectionPool.CONNECTION_READY,
+  ConnectionPool.CONNECTION_CLOSED,
+  ConnectionPool.CONNECTION_CHECK_OUT_STARTED,
+  ConnectionPool.CONNECTION_CHECK_OUT_FAILED,
+  ConnectionPool.CONNECTION_CHECKED_OUT,
+  ConnectionPool.CONNECTION_CHECKED_IN,
+  ConnectionPool.CONNECTION_POOL_CLEARED
+] as const;
 
 /**
  * An event indicating the start of a given