@webex/internal-plugin-mercury 2.60.0-next.8 → 2.60.1

package/dist/socket/socket.js.map

@@ -1 +1 @@
-{"version":3,"names":["_ws","_interopRequireDefault","require","_socketBase","Socket","getWebSocketConstructor","WS","_default","exports","default"],"sources":["socket.js"],"sourcesContent":["/*!\n * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.\n */\n\nimport WS from 'ws';\n\nimport Socket from './socket-base';\n\nSocket.getWebSocketConstructor = function getWebSocketConstructor() {\n  return WS;\n};\n\nexport default Socket;\n"],"mappings":";;;;;;;;AAIA,IAAAA,GAAA,GAAAC,sBAAA,CAAAC,OAAA;AAEA,IAAAC,WAAA,GAAAF,sBAAA,CAAAC,OAAA;AANA;AACA;AACA;;AAMAE,mBAAM,CAACC,uBAAuB,GAAG,SAASA,uBAAuBA,CAAA,EAAG;EAClE,OAAOC,WAAE;AACX,CAAC;AAAC,IAAAC,QAAA,GAAAC,OAAA,CAAAC,OAAA,GAEaL,mBAAM"}
+{"version":3,"names":["_ws","_interopRequireDefault","require","_socketBase","Socket","getWebSocketConstructor","WS","_default","exports","default"],"sources":["socket.js"],"sourcesContent":["/*!\n * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.\n */\n\nimport WS from 'ws';\n\nimport Socket from './socket-base';\n\nSocket.getWebSocketConstructor = function getWebSocketConstructor() {\n  return WS;\n};\n\nexport default Socket;\n"],"mappings":";;;;;;;;AAIA,IAAAA,GAAA,GAAAC,sBAAA,CAAAC,OAAA;AAEA,IAAAC,WAAA,GAAAF,sBAAA,CAAAC,OAAA;AANA;AACA;AACA;;AAMAE,mBAAM,CAACC,uBAAuB,GAAG,SAASA,uBAAuBA,CAAA,EAAG;EAClE,OAAOC,WAAE;AACX,CAAC;AAAC,IAAAC,QAAA,GAEaH,mBAAM;AAAAI,OAAA,CAAAC,OAAA,GAAAF,QAAA"}

package/dist/socket/socket.shim.js

@@ -32,5 +32,6 @@ _socketBase.default.getWebSocketConstructor = function getWebSocketConstructor()
   }
   return ws;
 };
-var _default = exports.default = _socketBase.default;
+var _default = _socketBase.default;
+exports.default = _default;
 //# sourceMappingURL=socket.shim.js.map

package/dist/socket/socket.shim.js.map

@@ -1 +1 @@
-{"version":3,"names":["_socketBase","_interopRequireDefault","require","Socket","getWebSocketConstructor","ws","WebSocket","MozWebSocket","global","window","self","_default","exports","default"],"sources":["socket.shim.js"],"sourcesContent":["/* eslint-disable no-restricted-globals */\n\n/*!\n * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.\n */\n\n/* eslint-env browser */\n\nimport Socket from './socket-base';\n\nSocket.getWebSocketConstructor = function getWebSocketConstructor() {\n  // Grabed from https://github.com/heineiuo/isomorphic-ws/blob/9b977394ac875638c045fd9cf774ed418484b394/browser.js\n  let ws;\n\n  if (typeof WebSocket !== 'undefined') {\n    ws = WebSocket;\n  } else if (typeof MozWebSocket !== 'undefined') {\n    // eslint-disable-next-line no-undef\n    ws = MozWebSocket;\n  } else if (typeof global !== 'undefined') {\n    ws = global.WebSocket || global.MozWebSocket;\n  } else if (typeof window !== 'undefined') {\n    ws = window.WebSocket || window.MozWebSocket;\n  } else if (typeof self !== 'undefined') {\n    ws = self.WebSocket || self.MozWebSocket;\n  }\n\n  return ws;\n};\n\nexport default Socket;\n"],"mappings":";;;;;;;;AAQA,IAAAA,WAAA,GAAAC,sBAAA,CAAAC,OAAA;AARA;;AAEA;AACA;AACA;;AAEA;;AAIAC,mBAAM,CAACC,uBAAuB,GAAG,SAASA,uBAAuBA,CAAA,EAAG;EAClE;EACA,IAAIC,EAAE;EAEN,IAAI,OAAOC,SAAS,KAAK,WAAW,EAAE;IACpCD,EAAE,GAAGC,SAAS;EAChB,CAAC,MAAM,IAAI,OAAOC,YAAY,KAAK,WAAW,EAAE;IAC9C;IACAF,EAAE,GAAGE,YAAY;EACnB,CAAC,MAAM,IAAI,OAAOC,MAAM,KAAK,WAAW,EAAE;IACxCH,EAAE,GAAGG,MAAM,CAACF,SAAS,IAAIE,MAAM,CAACD,YAAY;EAC9C,CAAC,MAAM,IAAI,OAAOE,MAAM,KAAK,WAAW,EAAE;IACxCJ,EAAE,GAAGI,MAAM,CAACH,SAAS,IAAIG,MAAM,CAACF,YAAY;EAC9C,CAAC,MAAM,IAAI,OAAOG,IAAI,KAAK,WAAW,EAAE;IACtCL,EAAE,GAAGK,IAAI,CAACJ,SAAS,IAAII,IAAI,CAACH,YAAY;EAC1C;EAEA,OAAOF,EAAE;AACX,CAAC;AAAC,IAAAM,QAAA,GAAAC,OAAA,CAAAC,OAAA,GAEaV,mBAAM"}
+{"version":3,"names":["_socketBase","_interopRequireDefault","require","Socket","getWebSocketConstructor","ws","WebSocket","MozWebSocket","global","window","self","_default","exports","default"],"sources":["socket.shim.js"],"sourcesContent":["/* eslint-disable no-restricted-globals */\n\n/*!\n * Copyright (c) 2015-2020 Cisco Systems, Inc. See LICENSE file.\n */\n\n/* eslint-env browser */\n\nimport Socket from './socket-base';\n\nSocket.getWebSocketConstructor = function getWebSocketConstructor() {\n  // Grabed from https://github.com/heineiuo/isomorphic-ws/blob/9b977394ac875638c045fd9cf774ed418484b394/browser.js\n  let ws;\n\n  if (typeof WebSocket !== 'undefined') {\n    ws = WebSocket;\n  } else if (typeof MozWebSocket !== 'undefined') {\n    // eslint-disable-next-line no-undef\n    ws = MozWebSocket;\n  } else if (typeof global !== 'undefined') {\n    ws = global.WebSocket || global.MozWebSocket;\n  } else if (typeof window !== 'undefined') {\n    ws = window.WebSocket || window.MozWebSocket;\n  } else if (typeof self !== 'undefined') {\n    ws = self.WebSocket || self.MozWebSocket;\n  }\n\n  return ws;\n};\n\nexport default Socket;\n"],"mappings":";;;;;;;;AAQA,IAAAA,WAAA,GAAAC,sBAAA,CAAAC,OAAA;AARA;;AAEA;AACA;AACA;;AAEA;;AAIAC,mBAAM,CAACC,uBAAuB,GAAG,SAASA,uBAAuBA,CAAA,EAAG;EAClE;EACA,IAAIC,EAAE;EAEN,IAAI,OAAOC,SAAS,KAAK,WAAW,EAAE;IACpCD,EAAE,GAAGC,SAAS;EAChB,CAAC,MAAM,IAAI,OAAOC,YAAY,KAAK,WAAW,EAAE;IAC9C;IACAF,EAAE,GAAGE,YAAY;EACnB,CAAC,MAAM,IAAI,OAAOC,MAAM,KAAK,WAAW,EAAE;IACxCH,EAAE,GAAGG,MAAM,CAACF,SAAS,IAAIE,MAAM,CAACD,YAAY;EAC9C,CAAC,MAAM,IAAI,OAAOE,MAAM,KAAK,WAAW,EAAE;IACxCJ,EAAE,GAAGI,MAAM,CAACH,SAAS,IAAIG,MAAM,CAACF,YAAY;EAC9C,CAAC,MAAM,IAAI,OAAOG,IAAI,KAAK,WAAW,EAAE;IACtCL,EAAE,GAAGK,IAAI,CAACJ,SAAS,IAAII,IAAI,CAACH,YAAY;EAC1C;EAEA,OAAOF,EAAE;AACX,CAAC;AAAC,IAAAM,QAAA,GAEaR,mBAAM;AAAAS,OAAA,CAAAC,OAAA,GAAAF,QAAA"}

package/package.json

@@ -1,5 +1,6 @@
 {
   "name": "@webex/internal-plugin-mercury",
+  "version": "2.60.1",
   "description": "",
   "license": "MIT",
   "main": "dist/index.js",
@@ -10,7 +11,7 @@
     "directory": "packages/@webex/internal-plugin-mercury"
   },
   "engines": {
-    "node": ">=16"
+    "node": ">=14"
   },
   "browser": {
     "./dist/socket/socket.js": "./dist/socket/socket.shim.js",
@@ -25,31 +26,31 @@
   "devDependencies": {
     "@babel/core": "^7.17.10",
     "@sinonjs/fake-timers": "^6.0.1",
-    "@webex/babel-config-legacy": "0.0.0",
-    "@webex/eslint-config-legacy": "0.0.0",
-    "@webex/jest-config-legacy": "0.0.0",
-    "@webex/legacy-tools": "0.0.0",
-    "@webex/test-helper-chai": "2.60.0-next.8",
-    "@webex/test-helper-mocha": "2.60.0-next.8",
-    "@webex/test-helper-mock-webex": "2.60.0-next.8",
-    "@webex/test-helper-test-users": "2.60.0-next.8",
+    "@webex/babel-config-legacy": "2.60.1",
+    "@webex/eslint-config-legacy": "2.60.1",
+    "@webex/jest-config-legacy": "2.60.1",
+    "@webex/legacy-tools": "2.60.1",
+    "@webex/test-helper-chai": "2.60.1",
+    "@webex/test-helper-mocha": "2.60.1",
+    "@webex/test-helper-mock-webex": "2.60.1",
+    "@webex/test-helper-test-users": "2.60.1",
     "eslint": "^8.24.0",
     "prettier": "^2.7.1",
     "sinon": "^9.2.4"
   },
   "dependencies": {
-    "@webex/common": "2.60.0-next.8",
-    "@webex/common-timers": "2.60.0-next.8",
-    "@webex/internal-plugin-device": "2.60.0-next.8",
-    "@webex/internal-plugin-feature": "2.60.0-next.8",
-    "@webex/internal-plugin-metrics": "2.60.0-next.8",
-    "@webex/test-helper-chai": "2.60.0-next.8",
-    "@webex/test-helper-mocha": "2.60.0-next.8",
-    "@webex/test-helper-mock-web-socket": "2.60.0-next.8",
-    "@webex/test-helper-mock-webex": "2.60.0-next.8",
-    "@webex/test-helper-refresh-callback": "2.60.0-next.8",
-    "@webex/test-helper-test-users": "2.60.0-next.8",
-    "@webex/webex-core": "2.60.0-next.8",
+    "@webex/common": "2.60.1",
+    "@webex/common-timers": "2.60.1",
+    "@webex/internal-plugin-device": "2.60.1",
+    "@webex/internal-plugin-feature": "2.60.1",
+    "@webex/internal-plugin-metrics": "2.60.1",
+    "@webex/test-helper-chai": "2.60.1",
+    "@webex/test-helper-mocha": "2.60.1",
+    "@webex/test-helper-mock-web-socket": "2.60.1",
+    "@webex/test-helper-mock-webex": "2.60.1",
+    "@webex/test-helper-refresh-callback": "2.60.1",
+    "@webex/test-helper-test-users": "2.60.1",
+    "@webex/webex-core": "2.60.1",
     "backoff": "^2.5.0",
     "lodash": "^4.17.21",
     "uuid": "^3.3.2",
@@ -58,12 +59,10 @@
   "scripts": {
     "build": "yarn build:src",
     "build:src": "webex-legacy-tools build -dest \"./dist\" -src \"./src\" -js -ts -maps",
-    "deploy:npm": "yarn npm publish",
     "test": "yarn test:style && yarn test:unit && yarn test:integration && yarn test:browser",
     "test:browser:broken": "webex-legacy-tools test --integration --unit --runner karma",
     "test:integration:broken": "webex-legacy-tools test --integration --runner mocha",
     "test:style": "eslint ./src/**/*.*",
     "test:unit": "webex-legacy-tools test --unit --runner jest"
-  },
-  "version": "2.60.0-next.8"
-}
+  }
+}

package/src/mercury.js

@@ -49,7 +49,7 @@ const Mercury = WebexPlugin.extend({
   @oneFlight
   connect(webSocketUrl) {
     if (this.connected) {
-      this.logger.info(`${this.namespace}: already connected, will not connect again`);
+      this.logger.info('mercury: already connected, will not connect again');
 
       return Promise.resolve();
     }
@@ -59,7 +59,7 @@ const Mercury = WebexPlugin.extend({
     return Promise.resolve(
       this.webex.internal.device.registered || this.webex.internal.device.register()
     ).then(() => {
-      this.logger.info(`${this.namespace}: connecting`);
+      this.logger.info('mercury: connecting');
 
       return this._connectWithBackoff(webSocketUrl);
     });
@@ -69,14 +69,16 @@ const Mercury = WebexPlugin.extend({
   disconnect() {
     return new Promise((resolve) => {
       if (this.backoffCall) {
-        this.logger.info(`${this.namespace}: aborting connection`);
+        this.logger.info('mercury: aborting connection');
         this.backoffCall.abort();
       }
 
       if (this.socket) {
         this.socket.removeAllListeners('message');
         this.once('offline', resolve);
-        resolve(this.socket.close());
+        this.socket.close();
+
+        return;
       }
 
       resolve();
@@ -163,7 +165,7 @@ const Mercury = WebexPlugin.extend({
     Promise.all([this._prepareUrl(socketUrl), this.webex.credentials.getUserToken()])
       .then(([webSocketUrl, token]) => {
         if (!this.backoffCall) {
-          const msg = `${this.namespace}: prevent socket open when backoffCall no longer defined`;
+          const msg = 'mercury: prevent socket open when backoffCall no longer defined';
 
           this.logger.info(msg);
 
@@ -183,7 +185,7 @@ const Mercury = WebexPlugin.extend({
 
         // if the consumer has supplied request options use them
         if (this.webex.config.defaultMercuryOptions) {
-          this.logger.info(`${this.namespace}: setting custom options`);
+          this.logger.info('mercury: setting custom options');
           options = {...options, ...this.webex.config.defaultMercuryOptions};
         }
 
@@ -191,8 +193,6 @@ const Mercury = WebexPlugin.extend({
         // the socket if it is in the process of being opened.
         this.socket = socket;
 
-        this.logger.info(`${this.namespace} connection url: ${webSocketUrl}`);
-
         return socket.open(webSocketUrl, options);
       })
       .then(() => {
@@ -201,7 +201,6 @@ const Mercury = WebexPlugin.extend({
             success: true,
           },
           tags: {
-            namespace: this.namespace,
             action: 'connected',
             url: attemptWSUrl,
           },
@@ -226,19 +225,19 @@ const Mercury = WebexPlugin.extend({
         if (reason.code !== 1006 && this.backoffCall && this.backoffCall.getNumRetries() > 0) {
           this._emit('connection_failed', reason, {retries: this.backoffCall.getNumRetries()});
         }
-        this.logger.info(`${this.namespace}: connection attempt failed`, reason);
+        this.logger.info('mercury: connection attempt failed', reason);
         // UnknownResponse is produced by IE for any 4XXX; treated it like a bad
         // web socket url and let WDM handle the token checking
         if (reason instanceof UnknownResponse) {
           this.logger.info(
-            `${this.namespace}: received unknown response code, refreshing device registration`
+            'mercury: received unknown response code, refreshing device registration'
           );
 
           return this.webex.internal.device.refresh().then(() => callback(reason));
         }
         // NotAuthorized implies expired token
         if (reason instanceof NotAuthorized) {
-          this.logger.info(`${this.namespace}: received authorization error, reauthorizing`);
+          this.logger.info('mercury: received authorization error, reauthorizing');
 
           return this.webex.credentials.refresh({force: true}).then(() => callback(reason));
         }
@@ -251,7 +250,7 @@ const Mercury = WebexPlugin.extend({
         // BadRequest implies current credentials are for a Service Account
         // Forbidden implies current user is not entitle for Webex
         if (reason instanceof BadRequest || reason instanceof Forbidden) {
-          this.logger.warn(`${this.namespace}: received unrecoverable response from mercury`);
+          this.logger.warn('mercury: received unrecoverable response from mercury');
           this.backoffCall.abort();
 
           return callback(reason);
@@ -262,14 +261,13 @@ const Mercury = WebexPlugin.extend({
             .then((haMessagingEnabled) => {
               if (haMessagingEnabled) {
                 this.logger.info(
-                  `${this.namespace}: received a generic connection error, will try to connect to another datacenter`
+                  'mercury: received a generic connection error, will try to connect to another datacenter'
                 );
                 this.webex.internal.metrics.submitClientMetrics('web-ha-mercury', {
                   fields: {
                     success: false,
                   },
                   tags: {
-                    namespace: this.namespace,
                     action: 'failed',
                     error: reason.message,
                     url: attemptWSUrl,
@@ -287,7 +285,7 @@ const Mercury = WebexPlugin.extend({
         return callback(reason);
       })
       .catch((reason) => {
-        this.logger.error(`${this.namespace}: failed to handle connection failure`, reason);
+        this.logger.error('mercury: failed to handle connection failure', reason);
         callback(reason);
       });
   },
@@ -303,9 +301,7 @@ const Mercury = WebexPlugin.extend({
         this.backoffCall = undefined;
         if (err) {
           this.logger.info(
-            `${
-              this.namespace
-            }: failed to connect after ${call.getNumRetries()} retries; log statement about next retry was inaccurate; ${err}`
+            `mercury: failed to connect after ${call.getNumRetries()} retries; log statement about next retry was inaccurate; ${err}`
           );
 
           return reject(err);
@@ -318,7 +314,7 @@ const Mercury = WebexPlugin.extend({
 
       // eslint-disable-next-line prefer-reflect
       call = backoff.call((callback) => {
-        this.logger.info(`${this.namespace}: executing connection attempt ${call.getNumRetries()}`);
+        this.logger.info(`mercury: executing connection attempt ${call.getNumRetries()}`);
         this._attemptConnection(webSocketUrl, callback);
       }, onComplete);
 
@@ -334,7 +330,7 @@ const Mercury = WebexPlugin.extend({
       }
 
       call.on('abort', () => {
-        this.logger.info(`${this.namespace}: connection aborted`);
+        this.logger.info('mercury: connection aborted');
         reject(new Error('Mercury Connection Aborted'));
       });
 
@@ -344,16 +340,16 @@ const Mercury = WebexPlugin.extend({
           const delay = Math.min(call.strategy_.nextBackoffDelay_, this.config.backoffTimeMax);
 
           this.logger.info(
-            `${this.namespace}: failed to connect; attempting retry ${number + 1} in ${delay} ms`
+            `mercury: failed to connect; attempting retry ${number + 1} in ${delay} ms`
           );
           /* istanbul ignore if */
           if (process.env.NODE_ENV === 'development') {
-            this.logger.debug(`${this.namespace}: `, err, err.stack);
+            this.logger.debug('mercury: ', err, err.stack);
           }
 
           return;
         }
-        this.logger.info(`${this.namespace}: connected`);
+        this.logger.info('mercury: connected');
       });
 
       call.start();
@@ -366,10 +362,7 @@ const Mercury = WebexPlugin.extend({
     try {
       this.trigger(...args);
     } catch (error) {
-      this.logger.error(`${this.namespace}: error occurred in event handler`, {
-        error,
-        arguments: args,
-      });
+      this.logger.error('mercury: error occurred in event handler', error);
     }
   },
 
@@ -410,20 +403,20 @@ const Mercury = WebexPlugin.extend({
         case 1003:
           // metric: disconnect
           this.logger.info(
-            `${this.namespace}: Mercury service rejected last message; will not reconnect: ${event.reason}`
+            `mercury: Mercury service rejected last message; will not reconnect: ${event.reason}`
           );
           this._emit('offline.permanent', event);
           break;
         case 4000:
           // metric: disconnect
-          this.logger.info(`${this.namespace}: socket replaced; will not reconnect`);
+          this.logger.info('mercury: socket replaced; will not reconnect');
           this._emit('offline.replaced', event);
           break;
         case 1001:
         case 1005:
         case 1006:
         case 1011:
-          this.logger.info(`${this.namespace}: socket disconnected; reconnecting`);
+          this.logger.info('mercury: socket disconnected; reconnecting');
           this._emit('offline.transient', event);
           this._reconnect(socketUrl);
           // metric: disconnect
@@ -431,25 +424,23 @@ const Mercury = WebexPlugin.extend({
           break;
         case 1000:
           if (normalReconnectReasons.includes(reason)) {
-            this.logger.info(`${this.namespace}: socket disconnected; reconnecting`);
+            this.logger.info('mercury: socket disconnected; reconnecting');
             this._emit('offline.transient', event);
             this._reconnect(socketUrl);
             // metric: disconnect
             // if (reason === done forced) metric: force closure
           } else {
-            this.logger.info(`${this.namespace}: socket disconnected; will not reconnect`);
+            this.logger.info('mercury: socket disconnected; will not reconnect');
             this._emit('offline.permanent', event);
           }
           break;
         default:
-          this.logger.info(
-            `${this.namespace}: socket disconnected unexpectedly; will not reconnect`
-          );
+          this.logger.info('mercury: socket disconnected unexpectedly; will not reconnect');
           // unexpected disconnect
           this._emit('offline.permanent', event);
       }
     } catch (error) {
-      this.logger.error(`${this.namespace}: error occurred in close handler`, error);
+      this.logger.error('mercury: error occurred in close handler', error);
     }
   },
 
@@ -457,7 +448,7 @@ const Mercury = WebexPlugin.extend({
     const envelope = event.data;
 
     if (process.env.ENABLE_MERCURY_LOGGING) {
-      this.logger.debug(`${this.namespace}: message envelope: `, envelope);
+      this.logger.debug('mercury: message envelope: ', envelope);
     }
 
     const {data} = envelope;
@@ -474,7 +465,7 @@ const Mercury = WebexPlugin.extend({
               resolve((this.webex[namespace] || this.webex.internal[namespace])[name](data))
             ).catch((reason) =>
               this.logger.error(
-                `${this.namespace}: error occurred in autowired event handler for ${data.eventType}`,
+                `mercury: error occurred in autowired event handler for ${data.eventType}`,
                 reason
               )
             );
@@ -493,12 +484,12 @@ const Mercury = WebexPlugin.extend({
         }
       })
       .catch((reason) => {
-        this.logger.error(`${this.namespace}: error occurred processing socket message`, reason);
+        this.logger.error('mercury: error occurred processing socket message', reason);
       });
   },
 
   _reconnect(webSocketUrl) {
-    this.logger.info(`${this.namespace}: reconnecting`);
+    this.logger.info('mercury: reconnecting');
 
     return this.connect(webSocketUrl);
   },

package/src/socket/socket-base.js

@@ -30,7 +30,6 @@ export default class Socket extends EventEmitter {
    */
   constructor() {
     super();
-    this._domain = 'unknown-domain';
     this.onmessage = this.onmessage.bind(this);
     this.onclose = this.onclose.bind(this);
   }
@@ -112,10 +111,10 @@ export default class Socket extends EventEmitter {
         return;
       }
       // logger is defined once open is called
-      this.logger.info(`socket,${this._domain}: closing`);
+      this.logger.info('socket: closing');
 
       if (socket.readyState === 2 || socket.readyState === 3) {
-        this.logger.info(`socket,${this._domain}: already closed`);
+        this.logger.info('socket: already closed');
         resolve();
 
         return;
@@ -135,7 +134,7 @@ export default class Socket extends EventEmitter {
 
       const closeTimer = safeSetTimeout(() => {
         try {
-          this.logger.info(`socket,${this._domain}: no close event received, forcing closure`);
+          this.logger.info('socket: no close event received, forcing closure');
           resolve(
             this.onclose({
               code: 1000,
@@ -143,12 +142,12 @@ export default class Socket extends EventEmitter {
             })
           );
         } catch (error) {
-          this.logger.warn(`socket,${this._domain}: force-close failed`, error);
+          this.logger.warn('socket: force-close failed', error);
         }
       }, this.forceCloseDelay);
 
       socket.onclose = (event) => {
-        this.logger.info(`socket,${this._domain}: close event fired`, event.code, event.reason);
+        this.logger.info('socket: close event fired', event.code, event.reason);
         clearTimeout(closeTimer);
         this.onclose(event);
         resolve(event);
@@ -172,12 +171,6 @@ export default class Socket extends EventEmitter {
    * @returns {Promise}
    */
   open(url, options) {
-    try {
-      this._domain = new URL(url).hostname;
-    } catch {
-      this._domain = url;
-    }
-
     return new Promise((resolve, reject) => {
       /* eslint complexity: [0] */
       if (!url) {
@@ -208,7 +201,7 @@ export default class Socket extends EventEmitter {
 
       const WebSocket = Socket.getWebSocketConstructor();
 
-      this.logger.info(`socket,${this._domain}: creating WebSocket`);
+      this.logger.info('socket: creating WebSocket');
       const socket = new WebSocket(url, [], options);
 
       socket.binaryType = 'arraybuffer';
@@ -216,7 +209,7 @@ export default class Socket extends EventEmitter {
 
       socket.onclose = (event) => {
         event = this._fixCloseCode(event);
-        this.logger.info(`socket,${this._domain}: closed before open`, event.code, event.reason);
+        this.logger.info('socket: closed before open', event.code, event.reason);
         switch (event.code) {
           case 1005:
             // IE 11 doesn't seem to allow 4XXX codes, so if we get a 1005, assume
@@ -238,10 +231,10 @@ export default class Socket extends EventEmitter {
       };
 
       socket.onopen = () => {
-        this.logger.info(`socket,${this._domain}: connected`);
+        this.logger.info('socket: connected');
         this._authorize()
           .then(() => {
-            this.logger.info(`socket,${this._domain}: authorized`);
+            this.logger.info('socket: authorized');
             socket.onclose = this.onclose;
             resolve();
           })
@@ -249,11 +242,11 @@ export default class Socket extends EventEmitter {
       };
 
       socket.onerror = (event) => {
-        this.logger.warn(`socket,${this._domain}: error event fired`, event);
+        this.logger.warn('socket: error event fired', event);
       };
 
       sockets.set(this, socket);
-      this.logger.info(`socket,${this._domain}: waiting for server`);
+      this.logger.info('socket: waiting for server');
     });
   }
 
@@ -263,7 +256,7 @@ export default class Socket extends EventEmitter {
    * @returns {undefined}
    */
   onclose(event) {
-    this.logger.info(`socket,${this._domain}: closed`, event.code, event.reason);
+    this.logger.info('socket: closed', event.code, event.reason);
     clearTimeout(this.pongTimer);
     clearTimeout(this.pingTimer);
 
@@ -285,10 +278,10 @@ export default class Socket extends EventEmitter {
       const data = JSON.parse(event.data);
       const sequenceNumber = parseInt(data.sequenceNumber, 10);
 
-      this.logger.debug(`socket,${this._domain}: sequence number: `, sequenceNumber);
+      this.logger.debug('socket: sequence number: ', sequenceNumber);
       if (this.expectedSequenceNumber && sequenceNumber !== this.expectedSequenceNumber) {
         this.logger.debug(
-          `socket,${this._domain}: sequence number mismatch indicates lost mercury message. expected: ${this.expectedSequenceNumber}, actual: ${sequenceNumber}`
+          `socket: sequence number mismatch indicates lost mercury message. expected: ${this.expectedSequenceNumber}, actual: ${sequenceNumber}`
         );
         this.emit('sequence-mismatch', sequenceNumber, this.expectedSequenceNumber);
       }
@@ -310,7 +303,7 @@ export default class Socket extends EventEmitter {
       // message from Mercury. At this time, the only action we have is to
       // ignore it and move on.
       /* istanbul ignore next */
-      this.logger.warn(`socket,${this._domain}: error while receiving WebSocket message`, error);
+      this.logger.warn('socket: error while receiving WebSocket message', error);
     }
   }
 
@@ -364,7 +357,7 @@ export default class Socket extends EventEmitter {
    */
   _authorize() {
     return new Promise((resolve) => {
-      this.logger.info(`socket,${this._domain}: authorizing`);
+      this.logger.info('socket: authorizing');
       this.send({
         id: uuid.v4(),
         type: 'authorization',
@@ -402,18 +395,12 @@ export default class Socket extends EventEmitter {
     if (event.code === 1005 && event.reason) {
       switch (event.reason.toLowerCase()) {
         case 'replaced':
-          this.logger.info(
-            `socket,${this._domain}: fixing CloseEvent code for reason: `,
-            event.reason
-          );
+          this.logger.info('socket: fixing CloseEvent code for reason: ', event.reason);
           event.code = 4000;
           break;
         case 'authentication failed':
         case 'authentication did not happen within the timeout window of 30000 seconds.':
-          this.logger.info(
-            `socket,${this._domain}: fixing CloseEvent code for reason: `,
-            event.reason
-          );
+          this.logger.info('socket: fixing CloseEvent code for reason: ', event.reason);
           event.code = 1008;
           break;
         default:
@@ -433,12 +420,10 @@ export default class Socket extends EventEmitter {
   _ping(id) {
     const confirmPongId = (event) => {
       try {
-        this.logger.debug(`socket,${this._domain}: pong`, event.data.id);
+        this.logger.debug('socket: pong', event.data.id);
         if (event.data && event.data.id !== id) {
-          this.logger.info(
-            `socket,${this._domain}: received pong for wrong ping id, closing socket`
-          );
-          this.logger.debug(`socket,${this._domain}: expected`, id, 'received', event.data.id);
+          this.logger.info('socket: received pong for wrong ping id, closing socket');
+          this.logger.debug('socket: expected', id, 'received', event.data.id);
           this.close({
             code: 1000,
             reason: 'Pong mismatch',
@@ -448,29 +433,24 @@ export default class Socket extends EventEmitter {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
-        this.logger.error(`socket,${this._domain}: error occurred in confirmPongId`, error);
+        this.logger.error('socket: error occurred in confirmPongId', error);
       }
     };
 
     const onPongNotReceived = () => {
       try {
-        this.logger.info(
-          `socket,${this._domain}: pong not receive in expected period, closing socket`
-        );
+        this.logger.info('socket: pong not receive in expected period, closing socket');
         this.close({
           code: 1000,
           reason: 'Pong not received',
         }).catch((reason) => {
-          this.logger.warn(
-            `socket,${this._domain}: failed to close socket after missed pong`,
-            reason
-          );
+          this.logger.warn('socket: failed to close socket after missed pong', reason);
         });
       } catch (error) {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
-        this.logger.error(`socket,${this._domain}: error occurred in onPongNotReceived`, error);
+        this.logger.error('socket: error occurred in onPongNotReceived', error);
       }
     };
 
@@ -482,10 +462,7 @@ export default class Socket extends EventEmitter {
         // This try/catch block was added as a debugging step; to the best of my
         // knowledge, the above can never throw.
         /* istanbul ignore next */
-        this.logger.error(
-          `socket,${this._domain}: error occurred in scheduleNextPingAndCancelPongTimer`,
-          error
-        );
+        this.logger.error('socket: error occurred in scheduleNextPingAndCancelPongTimer', error);
       }
     };
 
@@ -494,7 +471,7 @@ export default class Socket extends EventEmitter {
     this.once('pong', scheduleNextPingAndCancelPongTimer);
     this.once('pong', confirmPongId);
 
-    this.logger.debug(`socket,${this._domain}: ping ${id}`);
+    this.logger.debug(`socket: ping ${id}`);
 
     return this.send({
       id,

package/test/integration/spec/webex.js

@@ -32,13 +32,12 @@ describe('plugin-mercury', function () {
   );
 
   describe('onBeforeLogout()', () => {
-    it('disconnects the web socket', () => {
+    it('disconnects the web socket', () =>
       webex.logout({noRedirect: true}).then(() => {
         assert.called(webex.internal.mercury.disconnect);
         assert.isFalse(webex.internal.mercury.connected);
         assert.called(webex.internal.device.unregister);
         assert.isFalse(webex.internal.device.registered);
-      });
-    });
+      }));
   });
 });

package/test/unit/spec/mercury.js

@@ -558,7 +558,7 @@ describe('plugin-mercury', () => {
           return promiseTick(webex.internal.mercury.config.backoffTimeReset).then(() => {
             assert.equal(
               reason.message,
-              'Mercury: prevent socket open when backoffCall no longer defined'
+              'mercury: prevent socket open when backoffCall no longer defined'
             );
           });
         });
@@ -566,21 +566,12 @@ describe('plugin-mercury', () => {
     });
 
     describe('#_emit()', () => {
-      it('emits Error-safe events and log the error with the call parameters', () => {
-        const error = 'error';
-        const event = {data: 'some data'};
+      it('emits Error-safe events', () => {
         mercury.on('break', () => {
-          throw error;
+          throw new Error();
         });
-        sinon.stub(mercury.logger, 'error');
 
-        return Promise.resolve(mercury._emit('break', event)).then((res) => {
-          assert.calledWith(mercury.logger.error, 'Mercury: error occurred in event handler', {
-            error,
-            arguments: ['break', event],
-          });
-          return res;
-        });
+        return Promise.resolve(mercury._emit('break'));
       });
     });