Also mask properties of errors, fix #214

terehov · terehov · commit b20e22f8cf1e · 2023-02-18T00:26:25.000+02:00
diff --git a/examples/nodejs/index2.ts b/examples/nodejs/index2.ts
@@ -70,3 +70,24 @@ const loggerMap = new Logger({ name: 'mapLogger'});
 let map = new Map();
 map.set('foo', 'bar');
 loggerMap.debug('My Map: ', map) // prints in console "DEBUG myLogger My Map: {}"
+
+
+////////////////////////////
+
+const error = new TypeError();
+Object.assign(error, {
+  extensions: {
+    serviceName: 'upstream-service',
+    variables: {
+      firstName: 'foo',
+      phoneNumber: 'bar'
+    }
+  }
+});
+
+const log = new Logger({
+  maskValuesOfKeys: ["firstName", "phoneNumber"],
+  type: "json"
+})
+
+log.info(error);
diff --git a/src/BaseLogger.ts b/src/BaseLogger.ts
@@ -200,9 +200,7 @@ export class BaseLogger<LogObj> {
       seen.push(source);
     }
 
-    return isError(source)
-      ? source // dont copy Error
-      : isBuffer(source)
+    return isBuffer(source)
       ? source // dont copy Buffer
       : source instanceof Map ?
         new Map(source)
@@ -212,6 +210,14 @@ export class BaseLogger<LogObj> {
       ? source.map((item) => this._recursiveCloneAndMaskValuesOfKeys(item, keys, seen))
       : source instanceof Date
       ? new Date(source.getTime())
+      : isError(source)
+        ? Object.getOwnPropertyNames(source).reduce((o, prop) => {
+          // mask
+          o[prop] = keys.includes(this.settings?.maskValuesOfKeysCaseInsensitive !== true ? prop : prop.toLowerCase())
+            ? this.settings.maskPlaceholder
+            : this._recursiveCloneAndMaskValuesOfKeys((source as { [key: string]: unknown })[prop], keys, seen);
+          return o;
+        }, source)
       : source != null && typeof source === "object"
       ? Object.getOwnPropertyNames(source).reduce((o, prop) => {
           // mask
