feat(service-provider-server): log more detailed information about DNS lookups (#1191)

In particular, include whether os-dns-native made lookups
using the native OS DNS APIs or not.

Author: addaleax

packages/logging/package-lock.json

@@ -17,7 +17,7 @@
     "node": ">=12.4.0"
   },
   "dependencies": {
-    "@mongodb-js/devtools-connect": "^1.0.0",
+    "@mongodb-js/devtools-connect": "^1.1.1",
     "@mongosh/errors": "0.0.0-dev.0",
     "@mongosh/history": "0.0.0-dev.0",
     "@mongosh/types": "0.0.0-dev.0",

packages/logging/package.json

@@ -102,8 +102,17 @@ describe('setupLoggerAndTelemetry', () => {
     bus.emit('devtools-connect:connect-heartbeat-succeeded', { connectionId: 'localhost' });
     bus.emit('devtools-connect:connect-fail-early');
     bus.emit('devtools-connect:connect-attempt-finished');
-    bus.emit('devtools-connect:resolve-srv-error', { from: 'mongodb+srv://foo:[email protected]/', error: new Error('failed'), duringLoad: false });
-    bus.emit('devtools-connect:resolve-srv-succeeded', { from: 'mongodb+srv://foo:[email protected]/', to: 'mongodb://foo:[email protected]/' });
+    bus.emit('devtools-connect:resolve-srv-error', {
+      from: 'mongodb+srv://foo:[email protected]/',
+      error: new Error('failed'),
+      duringLoad: false,
+      resolutionDetails: []
+    });
+    bus.emit('devtools-connect:resolve-srv-succeeded', {
+      from: 'mongodb+srv://foo:[email protected]/',
+      to: 'mongodb://foo:[email protected]/',
+      resolutionDetails: []
+    });
     bus.emit('devtools-connect:missing-optional-dependency', { name: 'kerberos', error: new Error('no kerberos') });
     bus.emit('mongosh-sp:reset-connection-options');
 
@@ -201,9 +210,9 @@ describe('setupLoggerAndTelemetry', () => {
     expect(logOutput[i++].msg).to.equal('Aborting connection attempt as irrecoverable');
     expect(logOutput[i++].msg).to.equal('Connection attempt finished');
     expect(logOutput[i].msg).to.equal('Resolving SRV record failed');
-    expect(logOutput[i++].attr).to.deep.equal({ from: 'mongodb+srv://<credentials>@hello.world/', error: 'failed', duringLoad: false });
+    expect(logOutput[i++].attr).to.deep.equal({ from: 'mongodb+srv://<credentials>@hello.world/', error: 'failed', duringLoad: false, resolutionDetails: [] });
     expect(logOutput[i].msg).to.equal('Resolving SRV record succeeded');
-    expect(logOutput[i++].attr).to.deep.equal({ from: 'mongodb+srv://<credentials>@hello.world/', to: 'mongodb://<credentials>@db.hello.world/' });
+    expect(logOutput[i++].attr).to.deep.equal({ from: 'mongodb+srv://<credentials>@hello.world/', to: 'mongodb://<credentials>@db.hello.world/', resolutionDetails: [] });
     expect(logOutput[i].msg).to.equal('Missing optional dependency');
     expect(logOutput[i++].attr).to.deep.equal({ name: 'kerberos', error: 'no kerberos' });
     expect(logOutput[i++].msg).to.equal('Reconnect because of changed connection options');

packages/logging/src/setup-logger-and-telemetry.spec.ts

@@ -42,7 +42,8 @@ const config = {
     'mongodb-client-encryption': 'commonjs2 mongodb-client-encryption',
     kerberos: 'commonjs2 kerberos',
     snappy: 'commonjs2 snappy',
-    interruptor: 'commonjs2 interruptor'
+    interruptor: 'commonjs2 interruptor',
+    'os-dns-native': 'commonjs2 os-dns-native',
   }
 };
 

packages/node-runtime-worker-thread/webpack.config.js

@@ -38,7 +38,7 @@
     ]
   },
   "dependencies": {
-    "@mongodb-js/devtools-connect": "^1.0.0",
+    "@mongodb-js/devtools-connect": "^1.1.1",
     "@mongosh/errors": "0.0.0-dev.0",
     "@mongosh/service-provider-core": "0.0.0-dev.0",
     "@mongosh/types": "0.0.0-dev.0",

packages/service-provider-server/package-lock.json

@@ -32,6 +32,6 @@
     "unitTestsOnly": true
   },
   "dependencies": {
-    "@mongodb-js/devtools-connect": "^1.0.0"
+    "@mongodb-js/devtools-connect": "^1.1.1"
   }
 }