Skip to content

withSentry does not copy or generate _sentry-release-injection-file-*.js. Sometimes only copies .js.map file! #15291

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

Closed
3 tasks done
ian-pascoe opened this issue Feb 4, 2025 · 2 comments · Fixed by #15302
Closed
3 tasks done
Labels
Package: solidstart Issues related to the Sentry SolidStart SDK

Comments

@ian-pascoe
Copy link

ian-pascoe commented Feb 4, 2025

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/solidstart

SDK Version

8.54.0

Framework Version

@solidjs/start: 1.0.11

Link to Sentry event

No response

Reproduction Example/SDK Setup

I followed all the directions that were in the release notes for the withSentry solidstart change. (These should really be documented by the way, I only found these docs by digging and seeing an error in my build log)

Steps to Reproduce

  1. Set up solidstart project
  2. Set preset to bun
  3. Run build vinxi build within a docker container (this seems to be the key point)
    i. You may need to run the build a few times
  4. Observe that _sentry-release-injection-file-*.js is missing

You can see the example repo here: https://github.com/spirit-led-software/theaistudybible (the commit that has the issue is 78367144d1c6f7b8ddf0b65ba181ba664b83a17b)

Expected Result

Build should run successfully and the file should exist.

Actual Result

Without this file, my server refuses to start because of the bad import.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 4, 2025
@github-actions github-actions bot added the Package: solidstart Issues related to the Sentry SolidStart SDK label Feb 4, 2025
@s1gr1d
Copy link
Member

s1gr1d commented Feb 5, 2025

Thanks for sharing the repository. I created a PR which should fix this.

@ian-pascoe
Copy link
Author

ian-pascoe commented Feb 5, 2025

Thanks for sharing the repository. I created a PR which should fix this.

@s1gr1d thank you!! Will there be any documentation updates? The only place I could find docs on how to implement withSentry was in the release notes for 8.52.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: solidstart Issues related to the Sentry SolidStart SDK
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants