From e153ce2fdc911fdc1102c06029478f7e4082317f Mon Sep 17 00:00:00 2001 From: Luca Forstner Date: Wed, 5 Mar 2025 09:46:31 +0100 Subject: [PATCH 1/2] fix(core): Don't reverse values in event filters --- packages/core/src/integrations/eventFilters.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/core/src/integrations/eventFilters.ts b/packages/core/src/integrations/eventFilters.ts index a3fd2f5fb19b..fe1144c19499 100644 --- a/packages/core/src/integrations/eventFilters.ts +++ b/packages/core/src/integrations/eventFilters.ts @@ -211,9 +211,9 @@ function _getEventFilterUrl(event: Event): string | null { try { // If there are linked exceptions or exception aggregates we only want to match against the top frame of the "root" (the main exception) // The root always comes last in linked exceptions - const rootException = [...(event.exception?.values ?? []).reverse()]?.find( - value => value.mechanism?.parent_id === undefined && value.stacktrace?.frames?.length, - ); + const rootException = [...(event.exception?.values ?? [])] + .reverse() + ?.find(value => value.mechanism?.parent_id === undefined && value.stacktrace?.frames?.length); const frames = rootException?.stacktrace?.frames; return frames ? _getLastValidUrl(frames) : null; } catch (oO) { From f2775188a7b1b572e79728dcb3902f9d65a25249 Mon Sep 17 00:00:00 2001 From: Luca Forstner Date: Wed, 5 Mar 2025 09:48:25 +0100 Subject: [PATCH 2/2] rm unnecessary optional chain --- packages/core/src/integrations/eventFilters.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/core/src/integrations/eventFilters.ts b/packages/core/src/integrations/eventFilters.ts index fe1144c19499..fbf9475feca2 100644 --- a/packages/core/src/integrations/eventFilters.ts +++ b/packages/core/src/integrations/eventFilters.ts @@ -213,7 +213,7 @@ function _getEventFilterUrl(event: Event): string | null { // The root always comes last in linked exceptions const rootException = [...(event.exception?.values ?? [])] .reverse() - ?.find(value => value.mechanism?.parent_id === undefined && value.stacktrace?.frames?.length); + .find(value => value.mechanism?.parent_id === undefined && value.stacktrace?.frames?.length); const frames = rootException?.stacktrace?.frames; return frames ? _getLastValidUrl(frames) : null; } catch (oO) {