-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Netlify briefly broke esbuild builds, causing "Dynamic require of "X"" errors for every serverless rendered page [fixed] #4079
Comments
currently suspect this issue is related to SvelteKit 279->280 where there was an important breaking change nope.. 500 errors still happen with Kit 279 and Netlify Adapter 44: https://62163c6047aa970007eba63f--swyxkit.netlify.app/blog continuing to go back in versions. kit 270 and netlify 44 still has the error https://62163d105d832f0008263fe6--swyxkit.netlify.app/blog even kit 261 has the error https://62163d98e3c77e0007668c18--swyxkit.netlify.app/blog this is bad.. this means something broke in 260->261 and I don't understand why. |
i realized that my last working build on Feb 16 had Netlify adapter 45 and Kit 269 so i tried to repro that. somehow, it still doesn't work. i just found #2400 and trying their it still does not work: https://6216421075c7d30007e80be8--swyxkit.netlify.app/blog i am quite at a loss as to what to do. this error just keeps happening. |
i have put my failed attempts at debugging into a different branch, so as not to break current users https://github.com/sw-yx/swyxkit/commits/debuggingVersions |
Possibly related: #3978 (comment) and #4058. Does removing node_bundler help? |
I just tried with removing node_bundler, still the same |
confirmed that removing node bundler did not help :( any other options? |
I'm currently testing this, will let you know how it turns out. |
I think it's Netlify handling of node_modules |
created issue netlify/zip-it-and-ship-it#1036 |
Netlify reverted to working version of ZISI |
WOW so it was not my/our fault?!? amazing. thank you so much, will retry. edit: confirmed that a simple rebuild fixed it, now that Netlify reverted their change. watch netlify/zip-it-and-ship-it#1036 for when they try it again. |
@sw-yx no problem man, yeah, I was pretty sure it wasn't your/our problem, and I checked all the recent SvelteKit changes and there wasn't anything breaking, and when I tried to deploy it to Vercel, and it was working no issue, and the dev was working, so I figured the issue has to be something with Netlify, something with their build/packaging process. |
Esbuild source issue: evanw/esbuild#1921 |
Describe the bug
as part of regular maintenance for swyxkit i upgraded the deps as per the title
and now everything is throwing a 500 error with this
Dynamic require of "fs"
error, even though i'm pretty darn sure I don't actually write that code. EDIT: Possible related error: #2400this error only happens in prod build, not in local dev. is this something broken in the netlify adapter?
Reproduction
this is the netlify build logs https://app.netlify.com/sites/swyxkit/deploys/621638d27781070008fb992c
and this is the commit https://github.com/sw-yx/swyxkit/tree/c452df95219f73c4c3cd199490752656f0ab89ec
Logs
No response
System Info
Severity
blocking an upgrade
Additional Information
still trying to narrow down the precise version where this broke
The text was updated successfully, but these errors were encountered: