Merge pull request #98 from CyberSource/fix-external-logger-issues

Fixes issues in external logger configuration

Author: gnongsie

src/authentication/logging/ExternalLoggerWrapper.js

@@ -13,9 +13,7 @@ class ExternalLoggerWrapper{
     }
 
     isLoggerEmpty(){
-        if(this.extLogger === undefined)
-            return false;
-        return true;
+        return typeof this.extLogger === 'undefined'
     }
 }
 

src/authentication/logging/LogConfiguration.js

@@ -128,8 +128,8 @@ class LogConfiguration {
             this.hasExternalLogger = false;
         }
 
-        if((typeof (this.externalLogger) === "object" && !(this.externalLogger instanceof ExternalLoggerWrapper))
-            || this.externalLogger === undefined || !(this.externalLogger.isLoggerEmpty())){
+        if(this.hasExternalLogger == true && ((typeof (this.externalLogger) === "object" && !(this.externalLogger instanceof ExternalLoggerWrapper))
+            || this.externalLogger === undefined || (this.externalLogger.isLoggerEmpty()))){
             ApiException.LoggerException("No valid external logger object found. Turning off external logger flag.")
             this.hasExternalLogger = false;
         }