Assign .cause if not handled by super() call

hapijs · Dec 7, 2023 · 7c7f86c · 7c7f86c
1 parent b27692d
commit 7c7f86c
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -85,6 +85,10 @@ exports.Boom = class Boom extends Error {
         super(message ?? internals.codes.get(statusCode) ?? 'Unknown', { cause });
         Error.captureStackTrace(this, ctor);                           // Filter the stack to our external API
 
+        if (cause !== undefined) {
+            this.cause ??= cause;                                      // Explicitly assign cause to work with old runtimes
+        }
+
         internals.apply(this, data, statusCode, headers);
     }
 

diff --git a/test/index.js b/test/index.js
@@ -122,6 +122,27 @@ describe('Boom', () => {
         expect(err.output.payload.error).to.equal('Unknown');
     });
 
+    it('assigns a .cause property if Error does not support it', (flags) => {
+
+        const proto = Object.getPrototypeOf(Boom.Boom);
+        Object.setPrototypeOf(Boom.Boom, class extends Error {
+
+            constructor(message, _options) {
+
+                super(message);
+            }
+        });
+
+        flags.onCleanup = () => {
+
+            Object.setPrototypeOf(Boom.Boom, proto);
+        };
+
+        const err = new Boom.Boom('fail', { cause: 0 });
+        expect(err.cause).to.exist();
+        expect(err.cause).to.equal(0);
+    });
+
     describe('instanceof', () => {
 
         it('identifies a boom object', () => {