@nuxtjs/sentry - npm Package Compare versions

Comparing version 3.2.3 to 3.2.4

CHANGELOG.md

@@ -5,2 +5,9 @@ # Changelog
 
+### [3.2.4](https://github.com/nuxt-community/sentry-module/compare/v3.2.3...v3.2.4) (2020-02-14)
+
+
+### Bug Fixes
+
+* released client sourcemaps don't have correct prefix (no /_nuxt/) ([#155](https://github.com/nuxt-community/sentry-module/issues/155)) ([2c8bc83](https://github.com/nuxt-community/sentry-module/commit/2c8bc833d531ca843b4c9bc878ade5b1a49891db))
+
 ### [3.2.3](https://github.com/nuxt-community/sentry-module/compare/v3.2.2...v3.2.3) (2020-02-13)
@@ -7,0 +14,0 @@

lib/module.js

@@ -59,26 +59,33 @@ import { resolve, join } from 'path'
 
-  if (typeof options.webpackConfig.include === 'string') {
-    options.webpackConfig.include = [options.webpackConfig.include]
-  }
+  if (options.publishRelease) {
+    // Set urlPrefix to match resources on the client. That's not technically correct for the server
+    // source maps, but it is what it is for now.
+    const publicPath = join(this.options.router.base, this.options.build.publicPath)
+    options.webpackConfig.urlPrefix = publicPath.startsWith('/') ? `~${publicPath}` : publicPath
 
-  const { buildDir } = this.options
+    if (typeof options.webpackConfig.include === 'string') {
+      options.webpackConfig.include = [options.webpackConfig.include]
+    }
 
-  if (!options.disableServerRelease) {
-    options.webpackConfig.include.push(`${buildDir}/dist/server`)
-  }
-  if (!options.disableClientRelease) {
-    options.webpackConfig.include.push(`${buildDir}/dist/client`)
-  }
+    const { buildDir } = this.options
 
-  if (options.config.release && !options.webpackConfig.release) {
-    options.webpackConfig.release = options.config.release
-  }
+    if (!options.disableServerRelease) {
+      options.webpackConfig.include.push(`${buildDir}/dist/server`)
+    }
+    if (!options.disableClientRelease) {
+      options.webpackConfig.include.push(`${buildDir}/dist/client`)
+    }
 
-  if (options.attachCommits) {
-    options.webpackConfig.setCommits = {
-      auto: true
+    if (options.config.release && !options.webpackConfig.release) {
+      options.webpackConfig.release = options.config.release
     }
 
-    if (options.repo) {
-      options.webpackConfig.setCommits.repo = options.repo
+    if (options.attachCommits) {
+      options.webpackConfig.setCommits = {
+        auto: true
+      }
+
+      if (options.repo) {
+        options.webpackConfig.setCommits.repo = options.repo
+      }
     }
@@ -160,7 +167,2 @@ }
   if (!options.disabled) {
-    const { base } = this.options.router
-    const { publicPath } = this.options.build
-    const clientUrl = join(base, publicPath)
-    const clientUrlPrefix = clientUrl.startsWith('/') ? `~${clientUrl}` : clientUrl
-
     this.extendBuild((config, { isClient, isModern, isDev }) => {
@@ -170,2 +172,5 @@ if (!options.publishRelease || isDev) {
       }
+
+      config.devtool = 'source-map'
+
       // when not in spa mode upload only at server build
@@ -180,6 +185,2 @@ if (isClient && this.options.mode !== 'spa') {
 
-      config.devtool = 'source-map'
-
-      options.webpackConfig.urlPrefix = isClient ? clientUrlPrefix : '~/'
-
       config.plugins.push(new WebpackPlugin(options.webpackConfig))
@@ -186,0 +187,0 @@ logger.info('Enabling uploading of release sourcemaps to Sentry')

package.json

 {
   "name": "@nuxtjs/sentry",
-  "version": "3.2.3",
+  "version": "3.2.4",
   "description": "Sentry module for Nuxt.js",
@@ -5,0 +5,0 @@ "repository": "nuxt-community/sentry-module",

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 10 instances in 1 package

Fixed alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 10 instances in 1 package

Improved metrics

Total package byte prevSize

32067

1.53%

Lines of code

238

0.42%

No dependency changes