Merge pull request #62 from MinusGix/patch-2

Give greater insight into server errors for server owner

1 files changed, 13 insertions, 3 deletions

diff --git a/server/src/serverLib/CommandManager.js b/server/src/serverLib/CommandManager.js
index a99d0a9..0c4f0aa 100644
--- a/server/src/serverLib/CommandManager.js
+++ b/server/src/serverLib/CommandManager.js
@@ -23,6 +23,9 @@ class CommandManager {
     this.core = core;
     this.commands = [];
     this.categories = [];
+    if (!this.core.config.hasOwnProperty('logErrDetailed')) {
+      this.core.config.logErrDetailed = false;
+    }
   }
 
   /**
@@ -268,13 +271,20 @@ class CommandManager {
     try {
       return await command.run(this.core, server, socket, data);
     } catch (err) {
-      let errText = `Failed to execute '${command.info.name}': ${err}`;
-      console.log(errText);
+      let errText = `Failed to execute '${command.info.name}': `;
+      
+      // If we have more detail enabled, then we get the trace
+      // if it isn't, or the property doesn't exist, then we'll get only the message
+      if (this.core.config.logErrDetailed === true) {
+        console.log(errText + err.stack);
+      } else {
+        console.log(errText + err.toString())
+      }
 
       this.handleCommand(server, socket, {
         cmd: 'socketreply',
         cmdKey: server.cmdKey,
-        text: errText
+        text: errText + err.toString()
       });
 
       return null;