re-use getPossibleEventMessages util

Author: mydea

packages/core/src/baseclient.ts

@@ -52,6 +52,7 @@ import { SyncPromise, rejectedSyncPromise, resolvedSyncPromise } from './utils-h
 import { parseSampleRate } from './utils/parseSampleRate';
 import { prepareEvent } from './utils/prepareEvent';
 import { showSpanDropWarning } from './utils/spanUtils';
+import { getPossibleEventMessages } from './utils/eventUtils';
 
 const ALREADY_SEEN_ERROR = "Not capturing exception because it's already been captured.";
 
@@ -714,13 +715,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
    */
   protected _captureEvent(event: Event, hint: EventHint = {}, scope?: Scope): PromiseLike<string | undefined> {
     if (DEBUG_BUILD && isErrorEvent(event)) {
-      logger.log(
-        `Captured error event \`${
-          (event.exception && event.exception.values && event.exception.values[0] && event.exception.values[0].value) ||
-          event.message ||
-          '<unknown>'
-        }\``,
-      );
+      logger.log(`Captured error event \`${getPossibleEventMessages(event)[0] || '<unknown>'}\``);
     }
 
     return this._processEvent(event, hint, scope).then(

packages/core/src/integrations/inboundfilters.ts

@@ -5,6 +5,7 @@ import { defineIntegration } from '../integration';
 import { logger } from '../utils-hoist/logger';
 import { getEventDescription } from '../utils-hoist/misc';
 import { stringMatchesSomePattern } from '../utils-hoist/string';
+import { getPossibleEventMessages } from '../utils/eventUtils';
 
 // "Script error." is hard coded into browsers for errors that it can't read.
 // this is the result of a script being pulled in from an external domain and CORS.
@@ -117,7 +118,7 @@ function _isIgnoredError(event: Event, ignoreErrors?: Array<string | RegExp>): b
     return false;
   }
 
-  return _getPossibleEventMessages(event).some(message => stringMatchesSomePattern(message, ignoreErrors));
+  return getPossibleEventMessages(event).some(message => stringMatchesSomePattern(message, ignoreErrors));
 }
 
 function _isIgnoredTransaction(event: Event, ignoreTransactions?: Array<string | RegExp>): boolean {
@@ -147,32 +148,7 @@ function _isAllowedUrl(event: Event, allowUrls?: Array<string | RegExp>): boolea
   return !url ? true : stringMatchesSomePattern(url, allowUrls);
 }
 
-function _getPossibleEventMessages(event: Event): string[] {
-  const possibleMessages: string[] = [];
 
-  if (event.message) {
-    possibleMessages.push(event.message);
-  }
-
-  let lastException;
-  try {
-    // @ts-expect-error Try catching to save bundle size
-    lastException = event.exception.values[event.exception.values.length - 1];
-  } catch (e) {
-    // try catching to save bundle size checking existence of variables
-  }
-
-  if (lastException) {
-    if (lastException.value) {
-      possibleMessages.push(lastException.value);
-      if (lastException.type) {
-        possibleMessages.push(`${lastException.type}: ${lastException.value}`);
-      }
-    }
-  }
-
-  return possibleMessages;
-}
 
 function _isSentryError(event: Event): boolean {
   try {

packages/core/src/utils/eventUtils.ts

@@ -0,0 +1,31 @@
+import type { Event } from '../types-hoist';
+
+/**
+ * Get a list of possible event messages from a Sentry event.
+ */
+export function getPossibleEventMessages(event: Event): string[] {
+  const possibleMessages: string[] = [];
+
+  if (event.message) {
+    possibleMessages.push(event.message);
+  }
+
+  let lastException;
+  try {
+    // @ts-expect-error Try catching to save bundle size
+    lastException = event.exception.values[event.exception.values.length - 1];
+  } catch (e) {
+    // try catching to save bundle size checking existence of variables
+  }
+
+  if (lastException) {
+    if (lastException.value) {
+      possibleMessages.push(lastException.value);
+      if (lastException.type) {
+        possibleMessages.push(`${lastException.type}: ${lastException.value}`);
+      }
+    }
+  }
+
+  return possibleMessages;
+}