-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
@sentry/remix sourcemaps in production #5732
-
When running remix build --sourcemap you'll get following warning:
$ remix build --sourcemap Building Remix app in production mode... ⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️ You have enabled source maps in production. This will make your server-side code visible to the public and is highly discouraged! If you insist, please ensure you are using environment variables for secrets and not hard-coding them into your source! ⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️⚠️ Built in 501ms
This is a valid warning they are raising, shouldn't user be advised to remove the sourcemap after uploading it to Sentry with the CLI?
And if so can we add this as a suggestion in the docs?
Beta Was this translation helpful? Give feedback.
All reactions
Hey, thanks for writing in! Yes, we should advise users to do this. Transferring this discussion as an issue to backlog, but PRs are welcome for both docs and for the SDK.
Replies: 1 comment 1 reply
-
Hey, thanks for writing in! Yes, we should advise users to do this. Transferring this discussion as an issue to backlog, but PRs are welcome for both docs and for the SDK.
Beta Was this translation helpful? Give feedback.
All reactions
-
Done with #5740
Beta Was this translation helpful? Give feedback.