Skip to content

feat(core): Filter out error events with exception values and no stacktraces, values, or types #12387

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

To reduce noise, we want to filter out errors that do not provide any value as much as possible. We've seen error instances without message or stack trace which are totally useless so we should filter them out.

@lforst lforst force-pushed the lforst-no-stupid-erros branch from 89f87c7 to 3ebcb8c Compare June 6, 2024 12:54
Copy link
Contributor

github-actions bot commented Jun 6, 2024

size-limit report 📦

Path Size
@sentry/browser 22 KB (+0.34% 🔺)
@sentry/browser (incl. Tracing) 33.05 KB (+0.23% 🔺)
@sentry/browser (incl. Tracing, Replay) 68.65 KB (+0.16% 🔺)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 61.94 KB (+0.17% 🔺)
@sentry/browser (incl. Tracing, Replay with Canvas) 72.7 KB (+0.14% 🔺)
@sentry/browser (incl. Tracing, Replay, Feedback) 84.82 KB (+0.13% 🔺)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 86.66 KB (+0.13% 🔺)
@sentry/browser (incl. metrics) 26.18 KB (+0.3% 🔺)
@sentry/browser (incl. Feedback) 38.16 KB (+0.2% 🔺)
@sentry/browser (incl. sendFeedback) 26.59 KB (+0.28% 🔺)
@sentry/browser (incl. FeedbackAsync) 31.14 KB (+0.25% 🔺)
@sentry/react 24.77 KB (+0.3% 🔺)
@sentry/react (incl. Tracing) 36.07 KB (+0.21% 🔺)
@sentry/vue 26 KB (+0.3% 🔺)
@sentry/vue (incl. Tracing) 34.87 KB (+0.2% 🔺)
@sentry/svelte 22.13 KB (+0.34% 🔺)
CDN Bundle 23.35 KB (+0.21% 🔺)
CDN Bundle (incl. Tracing) 34.73 KB (+0.14% 🔺)
CDN Bundle (incl. Tracing, Replay) 68.67 KB (+0.09% 🔺)
CDN Bundle (incl. Tracing, Replay, Feedback) 73.82 KB (+0.08% 🔺)
CDN Bundle - uncompressed 68.6 KB (+0.28% 🔺)
CDN Bundle (incl. Tracing) - uncompressed 102.78 KB (+0.19% 🔺)
CDN Bundle (incl. Tracing, Replay) - uncompressed 212.71 KB (+0.11% 🔺)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 225.17 KB (+0.1% 🔺)
@sentry/nextjs (client) 35.44 KB (+0.22% 🔺)
@sentry/sveltekit (client) 33.68 KB (+0.23% 🔺)
@sentry/node 115.66 KB (+0.06% 🔺)
@sentry/node - without tracing 94.75 KB (+0.08% 🔺)
@sentry/aws-serverless 103.94 KB (+0.07% 🔺)

@lforst lforst merged commit a062912 into develop Jun 6, 2024
110 checks passed
@lforst lforst deleted the lforst-no-stupid-erros branch June 6, 2024 13:45
billyvg pushed a commit that referenced this pull request Jun 10, 2024
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