-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Next 13 Module build failed UnhandledSchemeError #8133
Comments
Assigning to @getsentry/support for routing, due by Tuesday, May 16th at 5:00 pm (sfo). ⏲️ |
Hey @alikleit, thanks for writing in! Are you running this on windows? If you run git status after you've run Also, after you've run
|
Ah what also might be a thing is that we need to detect if an src directory is being used and put our example project in there instead of using top level pages. |
Hey @AbhiPrasad! I'm running on windows, there's no git on the test project which I composed to make sure it's not a third party package on the actual project that is causing that issue. Same error with manual setup.. However, this is the log from the wizard:
|
Just want to say I have the exactly the same error which I had raised in StackOverflow: |
Hm, I think a somewhat recent change introduced this bug. Lemme check. |
Confirmed this on Windows. I'll check the potential fix! |
Thanks @timfish! |
That PR doesn't appear to fix it. Here is the full error. I guess the paths aren't displayed correctly in this error because the back slash is an escape character!
|
@timfish could you check out the latest commit and share the logs of the build before it crashes if you find the time? Thank you! |
Here is the output: TEST LOG {
sentryConfigFilePath: 'C:\\Users\\tim\\Documents\\Repositories\\next-sentry-app\\sentry.server.config.ts',
rp: 'C:\\Users\\tim\\Documents\\Repositories\\next-sentry-app\\src\\app\\layout.tsx',
imp: '..\\..\\..\\sentry.server.config.ts'
}
TEST LOG {
sentryConfigFilePath: 'C:\\Users\\tim\\Documents\\Repositories\\next-sentry-app\\sentry.server.config.ts',
rp: 'C:\\Users\\tim\\Documents\\Repositories\\next-sentry-app\\src\\app\\page.tsx',
imp: '..\\..\\..\\sentry.server.config.ts'
} After the error we see:
So it looks like there are two issues:
Like: templateCode = `import "${path.relative(path.dirname(this.resourcePath), sentryConfigFilePath).replace(/\\/g, '/')}";\n`.concat(templateCode); |
Due to the incremental builds I spent forever trying to work out why I was not getting any console output after the first initial build! I don't mind submitting a PR but wasn't sure if this was correct. Would you want to just exclude the import entirely if it isn't an absolute path? if (sentryConfigFilePath && path.isAbsolute(this.resourcePath)) { |
Still getting this error on Next 13.4.19 Windows 10
|
@rortan134 what SDK version are you on? |
I am on ^7.64.0 |
@rortan134 opened a PR to fix: #8875 |
Hey @lforst, I just tested the release (7.66.0) and now it seems to load but I'm getting the following warning on console:
|
@rortan134 Do you have some kind of unconventional node_modules structure? Can you try purging your dependencies and reinstalling them? |
I'm using npm workspaces, is it not compatible? Just did a fresh install and same thing 😪 |
@rortan134 I don't know what might cause this. Can you share a small reproduction example we could use to debug this further? Thanks! |
Seems like #8977 found the reason |
Environment
self-hosted (https://develop.sentry.dev/self-hosted/)
Steps to Reproduce
npx create-next-app@latest
npx @sentry/wizard@latest -i nextjs
pnpm build
Expected Result
Build should be successfull
Actual Result
Failed to compile.
dependencies
Product Area
Issues
Link
No response
DSN
No response
Version
No response
The text was updated successfully, but these errors were encountered: