Skip to content

fix(OUT-3553): drop browser-extension noise from Sentry #196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
priosshrsth wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fix(OUT-3553): drop browser-extension noise from Sentry #196

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
133a576
fix(OUT-3553): drop browser-extension noise from Sentry
priosshrsth Apr 23, 2026
ac102c3
refactor(OUT-3553): drop postMessageToContentScript marker
priosshrsth Apr 23, 2026
e2ecc56
refactor(OUT-3553): narrow extension-noise filter to one marker
priosshrsth Apr 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions src/instrumentation-client.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,24 @@ import * as Sentry from '@sentry/nextjs'

const dsn = process.env.NEXT_PUBLIC_SENTRY_DSN

// Marker left behind by multichainWallet-family browser extension content
// scripts that log errors via console in our page context (no stack frame
// for `denyUrls` to filter on). See OUT-3553.
const EXTENSION_NOISE_MARKER = 'multichainWallet'

const isBrowserExtensionNoise = (event: Sentry.ErrorEvent): boolean => {
try {
const probe = JSON.stringify({
message: event.message,
exception: event.exception,
extra: event.extra,
})
return probe.includes(EXTENSION_NOISE_MARKER)
} catch {
return false
}
}

Sentry.init({
dsn,

Expand All @@ -28,6 +46,14 @@ Sentry.init({
// Enable sending user PII (Personally Identifiable Information)
// https://docs.sentry.io/platforms/javascript/guides/nextjs/configuration/options/#sendDefaultPii
sendDefaultPii: true,

// Drop events whose top stack frame is in a browser extension URL. Catches
// native exceptions thrown from extension-owned code.
denyUrls: [/^chrome-extension:\/\//i, /^moz-extension:\/\//i, /^safari(-web)?-extension:\/\//i],

// Catches extension content scripts that log via console in our page context
// (no stack frame) — see OUT-3553.
beforeSend: (event) => (isBrowserExtensionNoise(event) ? null : event),
})

export const onRouterTransitionStart = Sentry.captureRouterTransitionStart
Loading