@honeybadger-io/js 3.2.3 → 3.2.7

package/dist/server/honeybadger.js

@@ -488,7 +488,7 @@ function formatCGIData(vars, prefix) {
 var notifier = {
     name: 'honeybadger-js',
     url: 'https://github.com/honeybadger-io/honeybadger-js',
-    version: '3.2.3'
+    version: '3.2.7'
 };
 // Split at commas
 var TAG_SEPARATOR = /,/;
@@ -564,10 +564,6 @@ var Client = /** @class */ (function () {
     Client.prototype.notify = function (notice, name, extra) {
         if (name === void 0) { name = undefined; }
         if (extra === void 0) { extra = undefined; }
-        if (!this.config.apiKey) {
-            this.logger.warn('Unable to send error report: no API key has been configured');
-            return false;
-        }
         if (this.config.disabled) {
             this.logger.warn('Deprecation warning: instead of `disabled: true`, use `reportData: false` to explicitly disable Honeybadger reporting. (Dropping notice: honeybadger.js is disabled)');
             return false;
@@ -576,6 +572,10 @@ var Client = /** @class */ (function () {
             this.logger.debug('Dropping notice: honeybadger.js is in development mode');
             return false;
         }
+        if (!this.config.apiKey) {
+            this.logger.warn('Unable to send error report: no API key has been configured');
+            return false;
+        }
         notice = makeNotice(notice);
         if (name && !(typeof name === 'object')) {
             var n = String(name);
@@ -689,7 +689,8 @@ var Client = /** @class */ (function () {
                 revision: notice.revision,
                 hostname: this.config.hostname,
                 time: new Date().toUTCString()
-            }
+            },
+            details: notice.details || {}
         };
     };
     /** @internal */
@@ -829,31 +830,38 @@ function errorHandler(err, req, _res, next) {
 }
 function lambdaHandler(handler) {
     return function lambdaHandler(event, context, callback) {
+        // in the case of an async handler, the length of the handler will be less than 3 (no callback function).
+        // if this is the case, we have to explicitly call the callback function from the function we are returning.
+        // we don't have to do that if the handler has third callback function parameter,
+        // because it will be called directly from inside the handler.
+        var shouldInvokeCallbackExplicitly = handler.length < 3;
         // eslint-disable-next-line prefer-rest-params
         var args = arguments;
         var dom = domain__default['default'].create();
         // eslint-disable-next-line @typescript-eslint/no-this-alias
         var hb = this;
-        dom.on('error', function (err) {
-            hb.notify(err, {
+        var hbHandler = function (err) {
+            var willNotify = hb.notify(err, {
                 afterNotify: function () {
                     hb.clear();
                     callback(err);
                 }
             });
-        });
+            if (!willNotify) {
+                callback(err);
+            }
+        };
+        dom.on('error', hbHandler);
         dom.run(function () {
             process.nextTick(function () {
                 Promise.resolve(handler.apply(this, args))
-                    .then(function () { hb.clear(); })
-                    .catch(function (err) {
-                    hb.notify(err, {
-                        afterNotify: function () {
-                            hb.clear();
-                            callback(err);
-                        }
-                    });
-                });
+                    .then(function (res) {
+                    hb.clear();
+                    if (shouldInvokeCallbackExplicitly) {
+                        callback(null, res);
+                    }
+                })
+                    .catch(hbHandler);
             });
         });
     }.bind(this);

package/dist/server/types/core/types.d.ts

@@ -64,6 +64,7 @@ export interface Notice {
     environment?: string | undefined;
     revision?: string | undefined;
     tags: string | string[];
+    details: Record<string, Record<string, unknown>>;
     __breadcrumbs: BreadcrumbRecord[];
     afterNotify?: AfterNotifyHandler;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@honeybadger-io/js",
-  "version": "3.2.3",
+  "version": "3.2.7",
   "license": "MIT",
   "homepage": "https://github.com/honeybadger-io/honeybadger-js",
   "author": {
@@ -48,26 +48,26 @@
     "@rollup/plugin-node-resolve": "^13.0.4",
     "@rollup/plugin-replace": "^3.0.0",
     "@rollup/plugin-typescript": "^8.2.5",
-    "@types/jest": "^26.0.24",
-    "@typescript-eslint/eslint-plugin": "^4.29.0",
-    "@typescript-eslint/parser": "^4.29.0",
-    "axios": ">=0.21.1",
+    "@types/jest": "^27.0.2",
+    "@typescript-eslint/eslint-plugin": "^5.0.0",
+    "@typescript-eslint/parser": "^5.0.0",
+    "axios": ">=0.21.4",
     "concurrently": "^6.2.1",
-    "eslint": "^7.32.0",
-    "eslint-plugin-import": "^2.23.4",
-    "eslint-plugin-jest": "^24.4.0",
+    "eslint": "^8.0.0",
+    "eslint-plugin-import": "^2.24.1",
+    "eslint-plugin-jest": "^25.0.1",
     "eslint-plugin-promise": "^5.1.0",
     "express": "^4.17.1",
     "jest": "^27.0.6",
-    "nock": "^13.1.1",
-    "rollup": "^2.56.1",
+    "nock": "^13.1.2",
+    "rollup": "^2.56.2",
     "rollup-plugin-uglify": "^6.0.2",
-    "shipjs": "0.23.3",
+    "shipjs": "0.24.0",
     "sinon": "^11.1.2",
-    "supertest": "^6.1.4",
-    "ts-jest": "^27.0.4",
-    "tsd": "^0.17.0",
-    "tslib": "^2.3.0",
+    "supertest": "^6.1.6",
+    "ts-jest": "^27.0.5",
+    "tsd": "^0.18.0",
+    "tslib": "^2.3.1",
     "typescript": "^4.3.5"
   },
   "readmeFilename": "README.md",