feat(NODE-5549): add ability to provide crl file

Author: durran

.evergreen/config.in.yml

@@ -636,8 +636,6 @@ functions:
           export PROJECT_DIRECTORY="$(pwd)"
           export NODE_LTS_VERSION=${NODE_LTS_VERSION}
           export DRIVERS_TOOLS="${DRIVERS_TOOLS}"
-          export SSL_CA_FILE="${SSL_CA_FILE}"
-          export SSL_KEY_FILE="${SSL_KEY_FILE}"
           export MONGODB_URI="${MONGODB_URI}"
 
           bash ${PROJECT_DIRECTORY}/.evergreen/run-tls-tests.sh

.evergreen/config.yml

@@ -589,8 +589,6 @@ functions:
           export PROJECT_DIRECTORY="$(pwd)"
           export NODE_LTS_VERSION=${NODE_LTS_VERSION}
           export DRIVERS_TOOLS="${DRIVERS_TOOLS}"
-          export SSL_CA_FILE="${SSL_CA_FILE}"
-          export SSL_KEY_FILE="${SSL_KEY_FILE}"
           export MONGODB_URI="${MONGODB_URI}"
 
           bash ${PROJECT_DIRECTORY}/.evergreen/run-tls-tests.sh

.evergreen/run-tls-tests.sh

@@ -4,7 +4,8 @@ set -o errexit  # Exit the script with error if any of the commands fail
 
 source "${PROJECT_DIRECTORY}/.evergreen/init-node-and-npm-env.sh"
 
-export SSL_KEY_FILE="$DRIVERS_TOOLS/.evergreen/x509gen/client.pem"
-export SSL_CA_FILE="$DRIVERS_TOOLS/.evergreen/x509gen/ca.pem"
+export TLS_KEY_FILE="$DRIVERS_TOOLS/.evergreen/x509gen/client.pem"
+export TLS_CA_FILE="$DRIVERS_TOOLS/.evergreen/x509gen/ca.pem"
+export TLS_CRL_FILE="$DRIVERS_TOOLS/.evergreen/x509gen/crl.pem"
 
 npm run check:tls

src/connection_string.ts

@@ -1095,6 +1095,9 @@ export const OPTIONS = {
   tlsCAFile: {
     type: 'string'
   },
+  tlsCRLFile: {
+    type: 'string'
+  },
   tlsCertificateKeyFile: {
     type: 'string'
   },

src/mongo_client.ts

@@ -117,6 +117,8 @@ export interface MongoClientOptions extends BSONSerializeOptions, SupportedNodeC
   tlsCertificateKeyFilePassword?: string;
   /** Specifies the location of a local .pem file that contains the root certificate chain from the Certificate Authority. This file is used to validate the certificate presented by the mongod/mongos instance. */
   tlsCAFile?: string;
+  /** Specifies the location of a local .crl file that contains the client revokation list. */
+  tlsCRLFile?: string;
   /** Bypasses validation of the certificates presented by the mongod/mongos instance */
   tlsAllowInvalidCertificates?: boolean;
   /** Disables hostname validation of the certificate presented by the mongod/mongos instance. */
@@ -790,7 +792,7 @@ export interface MongoOptions
    * | nodejs native option  | driver spec equivalent option name            | driver option type |
    * |:----------------------|:----------------------------------------------|:-------------------|
    * | `ca`                  | `tlsCAFile`                                   | `string`           |
-   * | `crl`                 | N/A                                           | `string`           |
+   * | `crl`                 | `tlsCRLFile`                                  | `string`           |
    * | `cert`                | `tlsCertificateKeyFile`                       | `string`           |
    * | `key`                 | `tlsCertificateKeyFile`                       | `string`           |
    * | `passphrase`          | `tlsCertificateKeyFilePassword`               | `string`           |
@@ -814,8 +816,8 @@ export interface MongoOptions
    * `cert` and `key` fields will be undefined.
    */
   tls: boolean;
-
   tlsCAFile?: string;
+  tlsCRLFile?: string;
   tlsCertificateKeyFile?: string;
 
   /** @internal */

test/manual/tls_support.test.ts

@@ -8,18 +8,19 @@ import {
   MongoServerSelectionError
 } from '../mongodb';
 
-const REQUIRED_ENV = ['MONGODB_URI', 'SSL_KEY_FILE', 'SSL_CA_FILE'];
+const REQUIRED_ENV = ['MONGODB_URI', 'TLS_KEY_FILE', 'TLS_CA_FILE', 'TLS_CRL_FILE'];
 
 describe('TLS Support', function () {
   for (const key of REQUIRED_ENV) {
     if (process.env[key] == null) {
-      throw new Error(`skipping SSL tests, ${key} environment variable is not defined`);
+      throw new Error(`skipping TLS tests, ${key} environment variable is not defined`);
     }
   }
 
   const CONNECTION_STRING = process.env.MONGODB_URI as string;
-  const TLS_CERT_KEY_FILE = process.env.SSL_KEY_FILE as string;
-  const TLS_CA_FILE = process.env.SSL_CA_FILE as string;
+  const TLS_CERT_KEY_FILE = process.env.TLS_KEY_FILE as string;
+  const TLS_CA_FILE = process.env.TLS_CA_FILE as string;
+  const TLS_CRL_FILE = process.env.TLS_CRL_FILE as string;
   const tlsSettings = {
     tls: true,
     tlsCertificateKeyFile: TLS_CERT_KEY_FILE,
@@ -114,6 +115,29 @@ describe('TLS Support', function () {
     });
   });
 
+  context('when providing tlsCRLFile', () => {
+    context('when the file will revoke the certificate', () => {
+      let client: MongoClient;
+      beforeEach(() => {
+        client = new MongoClient(CONNECTION_STRING, {
+          tls: true,
+          tlsCAFile: TLS_CA_FILE,
+          tlsCRLFile: TLS_CRL_FILE,
+          serverSelectionTimeoutMS: 5000,
+          connectTimeoutMS: 5000
+        });
+      });
+      afterEach(async () => {
+        await client?.close();
+      });
+
+      it('throws a MongoServerSelectionError', async () => {
+        const err = await client.connect().catch(e => e);
+        expect(err).to.be.instanceOf(MongoServerSelectionError);
+      });
+    });
+  });
+
   context('when tlsCertificateKeyFile is provided, but tlsCAFile is missing', () => {
     let client: MongoClient;
     beforeEach(() => {

test/unit/connection_string.test.ts

@@ -438,6 +438,13 @@ describe('Connection String', function () {
     });
   });
 
+  context('when providing tlsCRLFile', function () {
+    it('sets the tlsCRLFile option', function () {
+      const options = parseOptions('mongodb://localhost/?tls=true&tlsCRLFile=path/to/file.crl');
+      expect(options.tlsCRLFile).to.equal('path/to/file.crl');
+    });
+  });
+
   context('when both tls and ssl options are provided', function () {
     context('when the options are provided in the URI', function () {
       context('when the options are equal', function () {

test/unit/mongo_client.test.js

@@ -35,6 +35,7 @@ describe('MongoOptions', function () {
     const options = parseOptions('mongodb://localhost:27017/?ssl=true', {
       tlsCertificateKeyFile: filename,
       tlsCAFile: filename,
+      tlsCRLFile: filename,
       tlsCertificateKeyFilePassword: 'tlsCertificateKeyFilePassword'
     });
     fs.unlinkSync(filename);
@@ -61,6 +62,7 @@ describe('MongoOptions', function () {
     expect(options).to.not.have.property('cert');
     expect(options).to.have.property('tlsCertificateKeyFile', filename);
     expect(options).to.have.property('tlsCAFile', filename);
+    expect(options).to.have.property('tlsCRLFile', filename);
     expect(options).has.property('passphrase', 'tlsCertificateKeyFilePassword');
     expect(options).has.property('tls', true);
   });