Skip to content
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

Webpack ChunkLoadErrors Filtered out #57458

Closed
Tracked by #59564
smeubank opened this issue Oct 4, 2023 · 6 comments · Fixed by #60885
Closed
Tracked by #59564

Webpack ChunkLoadErrors Filtered out #57458

smeubank opened this issue Oct 4, 2023 · 6 comments · Fixed by #60885

Comments

@smeubank
Copy link
Member

smeubank commented Oct 4, 2023

Problem Statement

ChunkLoadErrors are not new, but seem to be impacting NextJS devs working with AppDir disproportionately. This leads to a buggy page or just a white page, it seems.

The problem is these errors are not very actionable all the time, and can happen often which simply leads to consuming a lot of Sentry quota.

As a fix there is now an inbound filter on by default, good for quota, bad for visibilty. Is there way we can ensure users are still alerted to this problem, without killing their quota?

image

As seen in examples like this, Sentry plays an important role in telling developers that this problem exists

Solution Brainstorm

  1. some kind of auto-refund
  2. performance issue

Background

Was surfaced on twitter with link to error on NextjS

vercel/next.js#38507

@lforst lforst changed the title NextJS ChunkLoadErrors Filtered out Webpack ChunkLoadErrors Filtered out Oct 4, 2023
@getsantry
Copy link
Contributor

getsantry bot commented Oct 4, 2023

Assigning to @getsentry/support for routing ⏲️

@smeubank smeubank transferred this issue from getsentry/sentry-javascript Oct 4, 2023
@getsantry getsantry bot moved this to Waiting for: Support in GitHub Issues with 👀 Oct 4, 2023
@getsantry
Copy link
Contributor

getsantry bot commented Oct 4, 2023

Routing to @getsentry/product-owners-issues for triage ⏲️

@getsantry getsantry bot moved this from Waiting for: Support to Waiting for: Product Owner in GitHub Issues with 👀 Oct 4, 2023
@HazAT
Copy link
Member

HazAT commented Oct 4, 2023

Besides what @smeubank proposed -

We could try to group all these errors together - explain to users that they should capture the error in their app and refresh the page.
While at the same time pointing them to the inbound filter where they can enable it then in case they don't care to do this.

But since it's a valid error, this approach might maintain trust in Sentry and our reporting.

@HazAT
Copy link
Member

HazAT commented Nov 9, 2023

It could be a new Issue Type - where instead of us showing a Stacktrace, we explain what the error is:

"This error happens when your the users bundle isn't refreshed when you do a new deploy. To fix this capture this error in your application and prompt the user to refresh the page."

@scefali
Copy link
Contributor

scefali commented Nov 16, 2023

We also need a grouping rule to ensure that all ChunkLoadErrors get grouped together. We want to add more help text to help people debug the error and link to the filter to turn it on if they don't want to see those types of errors.

@scttcper
Copy link
Member

Action items:

  • Add a strict grouping rule for these errors
  • Messaging with:
    • A description of the error
    • A link to the inbound filter setting for that project

@jangjodi jangjodi self-assigned this Nov 20, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants