You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reacted by Razvan Soare, Elisei Nicolae, Thomas Ramé, Tom Beckett, momsse, Dale L. Jefferson, Florian Bellotti, Nick DeRobertis, John Datserakis, Steven Eubank and 1 more
Thanks for the quick reply. I think that the sentry.client.config and the sentry.server.config files never get called due to the webpack plugin not working with the latest nextjs version
Wouldn't be so sure @AbhiPrasad We've seen this problem with 7.20.1 too, just updated to the latest for the repro repo. Next13 adds a completely new folder structure which will definitely mess with the nextjs plugin https://nextjs.org/blog/next-13#new-app-directory-beta
The Next.js SDK currently doesn't support the app dir. That feature is still in beta and we will sit it out a bit until we can be confident that the instrumentation we add will not break users.
On my side everything was fine with the pages/ folder, I just wanted to migrate to app/ directory (by using appDir: true) to benefit from beta features but Sentry just errored in the CI/CD:
***:build: TypeError: Cannot read properties of undefined (reading 'replace')
***:build: at Object.escapeStringForRegex (xxxxxxxxxxxx/node_modules/.pnpm/@sentry+utils@7.21.1/node_modules/@sentry/utils/cjs/string.js:147:22)
***:build: at Object.newWebpackFunction [as webpack] (xxxxxxxxxxxx/node_modules/.pnpm/@sentry+nextjs@7.21.1_next@13.0.5+react@18.2.0/node_modules/@sentry/nextjs/cjs/config/webpack.js:91:38)
***:build: at Object.getBaseWebpackConfig [as default] (xxxxxxxxxxxx/node_modules/.pnpm/next@13.0.5_biqbaboplfbrettd7655fr4n2y/node_modules/next/dist/build/webpack-config.js:1426:32)
I totally understand there is some work to be planned, so if someone succeeds in a near future, please share it here (even if that's a custom patch for now). In the meantime I will disable Sentry.
Thank you,
Reacted by Dylan Gattey, Tom Beckett and Yordis Prieto
This issue has gone three weeks without activity. In another week, I will close it.
But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!
"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀
Reacted by Thomas Ramé, Guillaume Gelin, Yann KOËTH, Tim Fish, Lachlan Campbell, Murad, Thomas, Mike Di Domenico, Florian Bellotti, Nick DeRobertis and 4 more
Activity
AbhiPrasad commentedon Nov 24, 2022
Hi @cipriancaba - let's dig into why this is happening!
To help us debug, could you provide where you ran Sentry + Next (locally, server, Vercel, etc.). A reproduction would be even better!
If you provide
debug: trueas a option to yourSentry.initcall, what does that show? Do you see any logs that indicate some kind of error?Thanks!
cipriancaba commentedon Nov 24, 2022
Thanks for the quick reply. I think that the
sentry.client.configand thesentry.server.configfiles never get called due to the webpack plugin not working with the latest nextjs versionhere is a repro:
https://github.com/razvan-soare/sentry-next13-example
AbhiPrasad commentedon Nov 24, 2022
Ah yeah it's a dup of #6284 probably then, which we are fixing at the moment!
AbhiPrasad commentedon Nov 24, 2022
Closing as a duplicate!
cipriancaba commentedon Nov 24, 2022
Wouldn't be so sure @AbhiPrasad We've seen this problem with 7.20.1 too, just updated to the latest for the repro repo. Next13 adds a completely new folder structure which will definitely mess with the nextjs plugin https://nextjs.org/blog/next-13#new-app-directory-beta
lforst commentedon Nov 25, 2022
The Next.js SDK currently doesn't support the
appdir. That feature is still in beta and we will sit it out a bit until we can be confident that the instrumentation we add will not break users.sneko commentedon Nov 28, 2022
Interested too :)
On my side everything was fine with the
pages/folder, I just wanted to migrate toapp/directory (by usingappDir: true) to benefit from beta features but Sentry just errored in the CI/CD:I totally understand there is some work to be planned, so if someone succeeds in a near future, please share it here (even if that's a custom patch for now). In the meantime I will disable Sentry.
Thank you,
fix(test): migrate to "app" directory within next.js to allow tests a…
fix(test): migrate to "app" directory within next.js to allow tests a…
ramnes commentedon Dec 12, 2022
Definitely interested in
app/support as well.Do you think there could be any kind of workaround in the meantime?
github-actions commentedon Jan 3, 2023
This issue has gone three weeks without activity. In another week, I will close it.
But! If you comment or otherwise update it, I will reset the clock, and if you label it
Status: BacklogorStatus: In Progress, I will leave it alone ... forever!"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀
yordis commentedon Jan 3, 2023
keep
lforst commentedon Mar 13, 2023
Closing because error reporting should be stable now. Please follow our roadmap for more news on the app dir: #6726