Never serialize .cause when an Error

Instead rely on its content being appended to the message and the stack. This is an alternative fix to pinojs#94, replacing pinojs#105 and pinojs#108 + makes pinojs#109 not needed
voxpelli · Jun 13, 2022 · 8fd1809 · 8fd1809
1 parent 23f62f2
commit 8fd1809
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/err.js b/lib/err.js
@@ -63,9 +63,9 @@ function errSerializer (err) {
   for (const key in err) {
     if (_err[key] === undefined) {
       const val = err[key]
-      if (val instanceof Error && key !== 'cause') {
+      if (val instanceof Error) {
         /* eslint-disable no-prototype-builtins */
-        if (!val.hasOwnProperty(seen)) {
+        if (key !== 'cause' && !val.hasOwnProperty(seen)) {
           _err[key] = errSerializer(val)
         }
       } else {