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

Source maps info don't show in with-sentry example #8873

Closed
yantakus opened this issue Sep 27, 2019 · 3 comments · Fixed by #11701
Closed

Source maps info don't show in with-sentry example #8873

yantakus opened this issue Sep 27, 2019 · 3 comments · Fixed by #11701
Labels
examples Issue/PR related to examples good first issue Easy to fix issues, good for newcomers

Comments

@yantakus
Copy link

yantakus commented Sep 27, 2019

Cloned with-sentry example, provided DSN, as described in readme. Built the app in production mode, raised an exception. Result - error stack traces point to uglified files:

image

As I see in the code, it should provide source maps to sentry.

Any additional configuration needed to set up source maps? Or it's just a bug?

@Timer Timer added good first issue Easy to fix issues, good for newcomers help wanted examples Issue/PR related to examples labels Oct 18, 2019
@StarpTech
Copy link
Contributor

StarpTech commented Nov 18, 2019

Sentry will try to fetch the sourcemaps relatively to your hosted scripts. There are common pitfalls:

  • You use a different domain e.g CDN to host your files
  • You didn't enable the token mechanism
  • On SSR it can't work because nodejs doesn't respect sourcemaps
  • Sentry can't fetch the files in development localhost
  • You can inline your sourcemaps but this is not ideal since it expose your source code.

As a solution you can upload your source maps to sentry.

https://docs.sentry.io/platforms/javascript/sourcemaps/

@a14m
Copy link
Contributor

a14m commented Apr 4, 2020

I've tried uploading the source maps to sentry but it actually didn't work, could you give an example on how to upload the files to sentry?
More on the issue reported here:

Timer pushed a commit that referenced this issue Apr 13, 2020
* Fix the with-sentry-simple example configuration code

This fixes the following issues:
- #8873
- #11642

* Update the README with the info about uploading to sentry configurations

* Fix the typo in SENTRY_DNS -> SENTRY_DSN
@balazsorban44
Copy link
Member

This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@vercel vercel locked as resolved and limited conversation to collaborators Jan 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples good first issue Easy to fix issues, good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants