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

fix(nextjs): Use webpack module paths to attempt to resolve internal request async storage module #9100

Merged
merged 1 commit into from Sep 25, 2023

Conversation

lforst
Copy link
Member

@lforst lforst commented Sep 25, 2023

Attempts to fix #9092

I could never reproduce that issue but my suspicion for that error is that somewhere in the node module resolution tree there is a nextjs installation which will make next/dist/client/components/request-async-storage resolve.

We probably need to limit the module resolution search by what webpack itself is looking for. We can do this by looking at webpackConfig.resolve.modules which contains all the resolution paths.

This is not a 100% probability fix but something that is worth trying.

@lforst lforst merged commit 622186c into develop Sep 25, 2023
49 checks passed
@lforst lforst deleted the lforst-fix-nextjs-async-loading branch September 25, 2023 11:33
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.

Sentry doesn't work on Next 13.5
2 participants