If object is passed as disconnect reason, JSON.stringify it before logging

Author: jondubois

scserver.js

@@ -864,11 +864,11 @@ SCServer.prototype._passThroughAuthenticateMiddleware = function (options, cb) {
       if (callbackInvoked) {
         self.emit('warning', new InvalidActionError('Callback for ' + self.MIDDLEWARE_AUTHENTICATE + ' middleware was already invoked'));
       } else {
+        callbackInvoked = true;
         var isBadToken = false;
         if (results.length) {
           isBadToken = results[results.length - 1] || false;
         }
-        callbackInvoked = true;
         if (err) {
           if (err === true) {
             err = new SilentMiddlewareBlockedError('Action was silently blocked by ' + self.MIDDLEWARE_AUTHENTICATE + ' middleware', self.MIDDLEWARE_AUTHENTICATE);

scsocket.js

@@ -222,7 +222,17 @@ SCSocket.prototype._onSCClose = function (code, data) {
     if (!SCSocket.ignoreStatuses[code]) {
       var failureMessage;
       if (data) {
-        failureMessage = 'Socket connection failed: ' + data;
+        var reasonString;
+        if (typeof data == 'object') {
+          try {
+            reasonString = JSON.stringify(data);
+          } catch(error) {
+            reasonString = data.toString();
+          }
+        } else {
+          reasonString = data;
+        }
+        failureMessage = 'Socket connection failed: ' + reasonString;
       } else {
         failureMessage = 'Socket connection failed for unknown reasons';
       }