Allow usage on runtimes with no Error.captureStackTrace()

Author: kanongil

lib/index.js

@@ -83,7 +83,9 @@ exports.Boom = class Boom extends Error {
         const { statusCode = 500, data, headers, decorate, ctor = exports.Boom, cause } = options;
 
         super(message ?? internals.codes.get(statusCode) ?? 'Unknown', { cause });
-        Error.captureStackTrace(this, ctor);                           // Filter the stack to our external API
+        if (typeof Error.captureStackTrace === 'function') {           // Only use when available
+            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

test/index.js

@@ -1013,22 +1013,40 @@ describe('Boom', () => {
 
     describe('stack trace', () => {
 
+        const helpers = ['badRequest', 'unauthorized', 'forbidden', 'notFound', 'methodNotAllowed',
+            'notAcceptable', 'proxyAuthRequired', 'clientTimeout', 'conflict',
+            'resourceGone', 'lengthRequired', 'preconditionFailed', 'entityTooLarge',
+            'uriTooLong', 'unsupportedMediaType', 'rangeNotSatisfiable', 'expectationFailed',
+            'badData', 'preconditionRequired', 'tooManyRequests',
+
+            // 500s
+            'internal', 'notImplemented', 'badGateway', 'serverUnavailable',
+            'gatewayTimeout', 'badImplementation'
+        ];
+
         it('should omit lib', () => {
 
-            ['badRequest', 'unauthorized', 'forbidden', 'notFound', 'methodNotAllowed',
-                'notAcceptable', 'proxyAuthRequired', 'clientTimeout', 'conflict',
-                'resourceGone', 'lengthRequired', 'preconditionFailed', 'entityTooLarge',
-                'uriTooLong', 'unsupportedMediaType', 'rangeNotSatisfiable', 'expectationFailed',
-                'badData', 'preconditionRequired', 'tooManyRequests',
+            for (const helper of helpers) {
+                const err = Boom[helper]();
+                expect(err.stack).to.not.match(/\/lib\/index\.js/);
+            }
+        });
 
-                // 500s
-                'internal', 'notImplemented', 'badGateway', 'serverUnavailable',
-                'gatewayTimeout', 'badImplementation'
-            ].forEach((name) => {
+        it('should not crash when Error.captureStackTrace is missing', (flags) => {
 
-                const err = Boom[name]();
-                expect(err.stack).to.not.match(/\/lib\/index\.js/);
-            });
+            const captureStackTrace = Error.captureStackTrace;
+
+            for (const helper of helpers) {
+                try {
+                    Error.captureStackTrace = undefined;
+                    var err = Boom[helper]();
+                }
+                finally {
+                    Error.captureStackTrace = captureStackTrace;
+                }
+
+                expect(err.stack).to.match(/\/lib\/index\.js/);
+            }
         });
     });