-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Problem with Nuxt source maps generation #15160
Comments
Hi @adiantek @MirzaeiDev, thanks for filing this. I just had a look at it seems like setting the We'll have to dig deeper into why this is the case tho. @MirzaeiDev you shouldn't have to add the |
@Azirafel17 Thanks for sharing the screenshots. Can you share the link to the Sentry event? If you don't want to post it here, you can send it to me via mail (without whitespaces): sigrid . huemer [at] sentry . io |
Hello, Sigrid. |
Hello @Azirafel17, And about your "Unhandled Rejection" event in the screenshot: This is a synthetic event without an attached stack trace. Seems like you have a Promise that is not rejected with an error. If the Promise would throw an error when it is rejected, you would see a stack trace. But this is not related to source maps, since source maps are only needed when there is a stack trace. |
Hello @s1gr1d |
Hi @s1gr1d My event json file The error call options that I used: I can't achieve the result of displaying the error as shown in the screenshot below: |
About your code: It looks alright how you call it. And you are running the build output of your app? Can you maybe share a small reproduction example? Because I am not able to reproduce this unless I am in dev mode (which does not upload source maps). |
@s1gr1d, In the information_for_problem_analysis folder, under the parsing information (logs, screenshots, events, sourcemaps) |
@s1gr1d 🎉🎉🎉 You are beautiful! The problem was the type of application launch. Thank you so much, after 2 weeks I found peace. Can this information be added to the documentation, because this nuance is not obvious? |
Yes, that would be useful. However, it's already in the documentation here. |
Thanks again @s1gr1d 😊 Have a nice day) |
@adiantek @MirzaeiDev You originally commented on this issue. Do you still experience problems with source maps or were you able to resolve those issues? |
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 remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
…ap settings (#15859) There is a total of 6 settings in Nuxt where you can disable/enable your source maps. This is not only difficult to understand when writing and maintaining the logic for this. It may also be hard to debug stuff for users.  There is this discussion around Nuxt source maps: #15028 And this issue: #15160 A problem with the current setup was not only the difficulty to understand all of this but also it sometimes just overwrote stuff you didn't want it to. Like in the default case of Nuxt, `sourcemap.server` is `true`, but as `nitro.rollupConfig.output.sourcemap` is always undefined, Sentry would overwrite this setting to `'hidden'`, even though the source maps were enabled already. --- The only two relevant options in Nuxt are the [root-level sourcemap options](https://nuxt.com/docs/guide/going-further/debugging#sourcemaps). Those settings will propagate to nitro, vite and rollup. As we overwrite the source maps setting if the setting is undefined, only the basic Nuxt source map settings (linked above) are taken into account for that from now on.
…ap settings (#15859) There is a total of 6 settings in Nuxt where you can disable/enable your source maps. This is not only difficult to understand when writing and maintaining the logic for this. It may also be hard to debug stuff for users.  There is this discussion around Nuxt source maps: #15028 And this issue: #15160 A problem with the current setup was not only the difficulty to understand all of this but also it sometimes just overwrote stuff you didn't want it to. Like in the default case of Nuxt, `sourcemap.server` is `true`, but as `nitro.rollupConfig.output.sourcemap` is always undefined, Sentry would overwrite this setting to `'hidden'`, even though the source maps were enabled already. --- The only two relevant options in Nuxt are the [root-level sourcemap options](https://nuxt.com/docs/guide/going-further/debugging#sourcemaps). Those settings will propagate to nitro, vite and rollup. As we overwrite the source maps setting if the setting is undefined, only the basic Nuxt source map settings (linked above) are taken into account for that from now on.
Why was this issue closed? This issue hasn't been touched recently and no longer seems relevant. Please feel free to leave a comment if you think this issue should be reopened. |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/nuxt
SDK Version
8.51.0
Framework Version
Nuxt 3.15.2
Link to Sentry event
No response
Reproduction Example/SDK Setup
Steps to Reproduce
npx nuxi@latest init projectname
cd projectname
npx nuxi@latest init projectname
nuxt.config.ts
to upload only generated.map
(code from SDK Setup)npm run generate
Expected Result
Source maps from generated page should be uploaded. Sentry/nuxt uploads only from nuxt-server.
sentry-rollup-plugin
trying to upload source maps beforePrerendering 3 initial routes with crawler
.Actual Result
The text was updated successfully, but these errors were encountered: