Correctly handle falsy values passed to onunhandledrejection handlers in Tracekit

CHANGELOG.md

@@ -3,6 +3,8 @@
 ## Unreleased
 
 - [browser] feat: Use framesToPop for InvaliantViolations in React errors (#2204)
+- [browser] fix: Make sure that falsy values are captured in unhandledrejections (#2207)
+- [loader] fix: Loader should also retrigger falsy values as errors (#2207)
 
 ## 5.6.1
 

packages/browser/src/integrations/globalhandlers.ts

@@ -94,7 +94,7 @@ export class GlobalHandlers implements Integration {
         message.error && isString(message.error.message) ? message.error.message : 'No error message';
     }
 
-    if (stacktrace.mechanism === 'onunhandledrejection' && stacktrace.incomplete) {
+    if (stacktrace.mechanism === 'onunhandledrejection' && (stacktrace.incomplete || stacktrace.mode === 'failed')) {
       return this._eventFromIncompleteRejection(stacktrace, error);
     }
 
@@ -166,8 +166,8 @@ export class GlobalHandlers implements Integration {
     if (event.exception.values && event.exception.values[0]) {
       event.exception.values[0].mechanism = {
         data: {
-          incomplete: true,
           mode: stacktrace.mode,
+          ...(stacktrace.incomplete && { incomplete: stacktrace.incomplete }),
           ...(stacktrace.message && { message: stacktrace.message }),
           ...(stacktrace.name && { name: stacktrace.name }),
         },

packages/browser/src/loader.js

@@ -31,8 +31,8 @@
     // content.p = promise rejection
     // content.f = function call the Sentry
     if (
-      (content.e ||
-        content.p ||
+      ('e' in content ||
+        'p' in content ||
         (content.f && content.f.indexOf('capture') > -1) ||
         (content.f && content.f.indexOf('showReportDialog') > -1)) &&
       lazy
@@ -139,9 +139,9 @@
 
       // And now capture all previously caught exceptions
       for (var i = 0; i < data.length; i++) {
-        if (data[i].e && tracekitErrorHandler) {
+        if ('e' in data[i] && tracekitErrorHandler) {
           tracekitErrorHandler.apply(_window, data[i].e);
-        } else if (data[i].p && tracekitUnhandledRejectionHandler) {
+        } else if ('p' in data[i] && tracekitUnhandledRejectionHandler) {
           tracekitUnhandledRejectionHandler.apply(_window, [data[i].p]);
         }
       }

packages/browser/src/tracekit.ts

@@ -272,7 +272,12 @@ TraceKit._report = (function reportModuleWrapper() {
    * @see https://developer.mozilla.org/en-US/docs/Web/API/PromiseRejectionEvent
    */
   function _traceKitWindowOnUnhandledRejection(e: any) {
-    var err = e && typeof e.reason !== 'undefined' ? e.reason : e;
+    var err = e;
+    // You cannot itterate over non-objects, but we want to check
+    // for the existence in any value, not for the value itself
+    try {
+      err = e && 'reason' in e ? e.reason : e;
+    } catch (_oO) {}
     var stack = TraceKit._computeStackTrace(err);
     stack.mechanism = 'onunhandledrejection';
     _notifyHandlers(stack, true, err);
@@ -906,8 +911,8 @@ TraceKit._computeStackTrace = (function _computeStackTraceWrapper() {
 
     return {
       original: ex,
-      name: ex.name,
-      message: ex.message,
+      name: ex && ex.name,
+      message: ex && ex.message,
       mode: 'failed',
     };
   }