@grimen/mybad - npm Package Compare versions

Comparing version 0.3.1 to 0.3.2

package.json

 {
     "name": "@grimen/mybad",
-    "version": "0.3.1",
+    "version": "0.3.2",
     "description": "My friendly error base class - for Node/JavaScript.",
@@ -5,0 +5,0 @@ "keywords": [

src/errors.js

@@ -432,2 +432,7 @@
 
+                        if (functionName.includes('/')) {
+                            fileName = functionName
+                            functionName = null
+                        }
+
                         functionName = functionName && functionName.trim()
@@ -469,2 +474,3 @@ lineNumber = lineNumber && parseInt(lineNumber)
                     const source = stackframe.source
+                    const uri = file && `file://${file}:${line}:${column}`
 
@@ -477,2 +483,3 @@ const stackobject = {
                         source,
+                        uri,
                     }
@@ -479,0 +486,0 @@

test/errors.test.js

@@ -317,2 +317,33 @@ /* global describe test expect */
             expect(error.stackobjects[0].source).toEqual(undefined)
+            expect(error.stackobjects[0].uri).toEqual(null)
+
+            error.stack = `
+                Error: Oops!
+                    at /foo/src/router.js:34:11
+                    at Layer.handle [as handle_request] (/foo/node_modules/express/lib/router/layer.js:95:5)
+                    at next (/foo/node_modules/express/lib/router/route.js:137:13)
+                    at Route.dispatch (/foo/node_modules/express/lib/router/route.js:112:3)
+                    at Layer.handle [as handle_request] (/foo/node_modules/express/lib/router/layer.js:95:5)
+                    at /foo/node_modules/express/lib/router/index.js:281:22
+                    at Function.process_params (/foo/node_modules/express/lib/router/index.js:335:12)
+                    at next (/foo/node_modules/express/lib/router/index.js:275:10)
+                    at initialize (/foo/node_modules/passport/lib/middleware/initialize.js:53:5)
+                    at Layer.handle [as handle_request] (/foo/node_modules/express/lib/router/layer.js:95:5)
+                    at trim_prefix (/foo/node_modules/express/lib/router/index.js:317:13)
+                    at /foo/node_modules/express/lib/router/index.js:284:7
+                    at Function.process_params (/foo/node_modules/express/lib/router/index.js:335:12)
+                    at next (/foo/node_modules/express/lib/router/index.js:275:10)
+                    at initialize (/foo/node_modules/passport/lib/middleware/initialize.js:53:5)
+                    at /foo/src/middleware.js:34:20
+            `.replace(/^\s{16}/gmi, '')
+
+            expect(error.stackobjects).toBeType('array')
+            expect(error.stackobjects.length).toBeGreaterThan(0)
+            expect(error.stackobjects[0]).toBeType('object')
+            expect(error.stackobjects[0].column).toEqual(11)
+            expect(error.stackobjects[0].file).toEqual('/foo/src/router.js')
+            expect(error.stackobjects[0].function).toEqual(null)
+            expect(error.stackobjects[0].line).toEqual(34)
+            expect(error.stackobjects[0].source).toEqual(undefined)
+            expect(error.stackobjects[0].uri).toEqual(`file:///foo/src/router.js:34:11`)
         })
@@ -347,3 +378,3 @@
             expect(error.data.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(error.data.stack[0].line).toEqual(320)
+            expect(error.data.stack[0].line).toEqual(351)
             expect(error.data.stack[0].source).toEqual(undefined)
@@ -365,3 +396,3 @@
             expect(error.data.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(error.data.stack[0].line).toEqual(320)
+            expect(error.data.stack[0].line).toEqual(351)
             expect(error.data.stack[0].source).toEqual(undefined)
@@ -396,3 +427,3 @@ })
             expect(data.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(data.stack[0].line).toEqual(368)
+            expect(data.stack[0].line).toEqual(399)
             expect(data.stack[0].source).toEqual(undefined)
@@ -421,3 +452,3 @@
             expect(data.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(data.stack[0].line).toEqual(368)
+            expect(data.stack[0].line).toEqual(399)
             expect(data.stack[0].source).toEqual(undefined)
@@ -528,3 +559,3 @@ })
             expect(errorObject.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(errorObject.stack[0].line).toEqual(497)
+            expect(errorObject.stack[0].line).toEqual(528)
             expect(errorObject.stack[0].source).toEqual(undefined)
@@ -560,3 +591,3 @@
             expect(errorObject.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(errorObject.stack[0].line).toEqual(528)
+            expect(errorObject.stack[0].line).toEqual(559)
             expect(errorObject.stack[0].source).toEqual(undefined)
@@ -589,3 +620,3 @@
             expect(errorObject.stack[0].function).toEqual(expect.stringMatching(/^Object\.(?:test|<anonymous>|mybad\.Error)$/))
-            expect(errorObject.stack[0].line).toEqual(557)
+            expect(errorObject.stack[0].line).toEqual(588)
             expect(errorObject.stack[0].source).toEqual(undefined)
@@ -592,0 +623,0 @@ })

New alerts

Environment variable access

Supply chain risk

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

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

50680

increased by5.1%

Lines of code

949

increased by3.83%

Worsened metrics

Number of low supply chain risk alerts

20

increased by66.67%

No dependency changes