Skip to content

feat(core): Ignore additional common but inactionable errors #12384

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.

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

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Jun 6, 2024

Adds a few errors to our default ignore list that we have identified analyzing incoming events on Sentry:

  • undefined is not an object (evaluating 'a.L'): 2250 projects, 12598 issues
  • can\'t redefine non-configurable property "solana": 647 projects, 1600 issues
  • vv().getRestrictions is not a function. (In 'vv().getRestrictions(1,a)', 'vv().getRestrictions' is undefined): 612 projects, 1555 issues
  • Can't find variable: _AutofillCallbackHandler: 447 projects, 4135 issues

@lforst lforst requested review from HazAT, Lms24 and AbhiPrasad June 6, 2024 12:02
Copy link
Contributor

github-actions bot commented Jun 6, 2024

size-limit report 📦

Path Size
@sentry/browser 21.92 KB (+0.57% 🔺)
@sentry/browser (incl. Tracing) 32.97 KB (+0.39% 🔺)
@sentry/browser (incl. Tracing, Replay) 68.55 KB (+0.18% 🔺)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 61.84 KB (+0.2% 🔺)
@sentry/browser (incl. Tracing, Replay with Canvas) 72.6 KB (+0.18% 🔺)
@sentry/browser (incl. Tracing, Replay, Feedback) 84.71 KB (+0.16% 🔺)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 86.55 KB (+0.15% 🔺)
@sentry/browser (incl. metrics) 26.11 KB (+0.47% 🔺)
@sentry/browser (incl. Feedback) 38.09 KB (+0.34% 🔺)
@sentry/browser (incl. sendFeedback) 26.51 KB (+0.48% 🔺)
@sentry/browser (incl. FeedbackAsync) 31.06 KB (+0.41% 🔺)
@sentry/react 24.69 KB (+0.5% 🔺)
@sentry/react (incl. Tracing) 36 KB (+0.33% 🔺)
@sentry/vue 25.93 KB (+0.48% 🔺)
@sentry/vue (incl. Tracing) 34.8 KB (+0.36% 🔺)
@sentry/svelte 22.06 KB (+0.57% 🔺)
CDN Bundle 23.3 KB (+0.58% 🔺)
CDN Bundle (incl. Tracing) 34.68 KB (+0.36% 🔺)
CDN Bundle (incl. Tracing, Replay) 68.62 KB (+0.19% 🔺)
CDN Bundle (incl. Tracing, Replay, Feedback) 73.77 KB (+0.17% 🔺)
CDN Bundle - uncompressed 68.42 KB (+0.38% 🔺)
CDN Bundle (incl. Tracing) - uncompressed 102.6 KB (+0.25% 🔺)
CDN Bundle (incl. Tracing, Replay) - uncompressed 212.49 KB (+0.13% 🔺)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 224.96 KB (+0.12% 🔺)
@sentry/nextjs (client) 35.37 KB (+0.4% 🔺)
@sentry/sveltekit (client) 33.6 KB (+0.41% 🔺)
@sentry/node 115.59 KB (+0.1% 🔺)
@sentry/node - without tracing 94.68 KB (+0.12% 🔺)
@sentry/aws-serverless 103.87 KB (+0.12% 🔺)

/^ResizeObserver loop completed with undelivered notifications.$/, // The browser logs this when a ResizeObserver handler takes a bit longer. Usually this is not an actual issue though. It indicates slowness.
/^Cannot redefine property: googletag$/, // This is thrown when google tag manager is used in combination with an ad blocker
"undefined is not an object (evaluating 'a.L')", // Random error that happens but not actionable or noticeable to end-users.
'can\'t redefine non-configurable property "solana"', // Probably a browser extension or custom browser (Brave) throwing this error
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not 100% sure, this appears pretty specific, but no strong feelings, also OK to keep it here :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These things being specific is good :D. This particular error is in 647 projects, which is way too many to be actual application code. I am pretty sure this is brave doing things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants