Clone errors before masking and make fields writable, fix #217

examples/nodejs/index2.ts

@@ -94,3 +94,18 @@ const log = new Logger({
 log.info(error);
 
 ////////////////////////////
+
+function createReadonlyError(message: string, property: string) {
+  const error = new Error(message);
+  Object.defineProperty(error, "property", {
+    value: property,
+    writable: false,
+    enumerable: true,
+    configurable: false,
+  });
+  return error;
+}
+
+const e = createReadonlyError("message", "property");
+
+logger.error(e);

src/BaseLogger.ts

@@ -217,7 +217,7 @@ export class BaseLogger<LogObj> {
             ? this.settings.maskPlaceholder
             : this._recursiveCloneAndMaskValuesOfKeys((source as { [key: string]: unknown })[prop], keys, seen);
           return o;
-        }, source)
+        }, this._cloneError(source as Error))
       : source != null && typeof source === "object"
       ? Object.getOwnPropertyNames(source).reduce((o, prop) => {
           // mask
@@ -275,6 +275,21 @@ export class BaseLogger<LogObj> {
     return clonedLogObj;
   }
 
+  private _cloneError<T extends Error>(error: T): T {
+    const ErrorConstructor = error.constructor as new (message?: string) => T;
+    const newError = new ErrorConstructor(error.message);
+    Object.assign(newError, error);
+    const propertyNames = Object.getOwnPropertyNames(newError);
+    for (const propName of propertyNames) {
+      const propDesc = Object.getOwnPropertyDescriptor(newError, propName);
+      if (propDesc) {
+        propDesc.writable = true;
+        Object.defineProperty(newError, propName, propDesc);
+      }
+    }
+    return newError;
+  }
+
   private _toErrorObject(error: Error): IErrorObject {
     return {
       nativeError: error,