-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed as not planned
Closed as not planned
Copy link
Labels
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/browser
SDK Version
latest
Framework Version
Remix 8.35.0
Link to Sentry event
No response
Reproduction Example/SDK Setup
No response
Steps to Reproduce
- Use "Automatic Configuration" command
Expected Result
The changes would include the following code included in the manual setup documentation:
NODE_OPTIONS='--import=./instrument.server.mjs' remix-serve build
# or
NODE_OPTIONS='--require=./instrument.server.cjs' remix-serve build
or...
The "Automatic Configuration" docs would say if you should include these changes in your project.
It's not clear to me if I need to prefix my build commands with the code above, but it looks like I should.
Actual Result
From what I can tell, the instrument.server.mjs
file is not imported/used if you only use the automatic configuration.
Metadata
Metadata
Assignees
Labels
Projects
Status
Waiting for: Community