Skip to content

Commit

Permalink
ref(with-sentry example): Explicitly set hideSourceMaps (#40079)
Browse files Browse the repository at this point in the history
This adds a default value for `hideSourceMaps` to the `next.config.js` the `with-sentry` example app provides, so that new users will use `hidden-source-map` as their Webpack `devtool` by default in client-side builds. (This prevents browser devtools from seeing sourcemaps and therefore prevents original source code from showing up automatically in the `Sources` tab. (More information on `devtool` values can be found [here](https://webpack.js.org/configuration/devtool/).)

In conjunction with getsentry/sentry-javascript#5649, getsentry/sentry-wizard#188, and getsentry/sentry-docs#5464, this is the first step in addressing the concerns raised in getsentry/sentry-javascript#4489. See [here](getsentry/sentry-javascript#4489 (comment)) for more details.
  • Loading branch information
lobsterkatie committed Sep 5, 2022
1 parent 391f6a9 commit 77e7529
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions examples/with-sentry/next.config.js
Expand Up @@ -7,6 +7,16 @@ const { withSentryConfig } = require('@sentry/nextjs')

const moduleExports = {
// Your existing module.exports

sentry: {
// Use `hidden-source-map` rather than `source-map` as the Webpack `devtool`
// for client-side builds. (This will be the default starting in
// `@sentry/nextjs` version 8.0.0.) See
// https://webpack.js.org/configuration/devtool/ and
// https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#use-hidden-source-map
// for more information.
hideSourceMaps: true,
},
}

// Make sure adding Sentry options is the last code to run before exporting, to
Expand Down

0 comments on commit 77e7529

Please sign in to comment.