Comparing version 0.2.1 to 0.2.2
@@ -72,7 +72,8 @@ (function() { | ||
exports.format_stack = function(err, frames) { | ||
var lines; | ||
var e, lines; | ||
lines = []; | ||
try { | ||
lines.push(err.toString()); | ||
} catch (e) { | ||
} catch (_error) { | ||
e = _error; | ||
console.log('Caught error in longjohn. Please report this to matt.insler@gmail.com.'); | ||
@@ -117,3 +118,3 @@ } | ||
}); | ||
if (!(error.__previous__ != null) && in_prepare === 1) { | ||
if ((error.__previous__ == null) && in_prepare === 1) { | ||
error.__previous__ = current_trace_error; | ||
@@ -177,2 +178,3 @@ } | ||
new_callback = function() { | ||
var e; | ||
current_trace_error = trace_error; | ||
@@ -182,3 +184,4 @@ trace_error = null; | ||
return callback.apply(this, arguments); | ||
} catch (e) { | ||
} catch (_error) { | ||
e = _error; | ||
e.stack; | ||
@@ -321,2 +324,6 @@ throw e; | ||
if (process.env.NODE_ENV === 'production') { | ||
console.warn('NOTICE: Longjohn is known to cause CPU usage due to its extensive data collection during runtime.\nIt generally should not be used in production applications.'); | ||
} | ||
}).call(this); |
{ | ||
"name": "longjohn", | ||
"description": "Long stack traces for node.js inspired by https://github.com/tlrobinson/long-stack-traces", | ||
"version": "0.2.1", | ||
"version": "0.2.2", | ||
"homepage": "https://github.com/mattinsler/longjohn", | ||
@@ -29,4 +29,4 @@ "author": { | ||
"devDependencies": { | ||
"grunt": "~0.3.15", | ||
"grunt-contrib": "~0.3.0", | ||
"grunt": "~0.4.0", | ||
"grunt-contrib-coffee": "~0.7.0", | ||
"mocha": "latest", | ||
@@ -33,0 +33,0 @@ "coffee-script": "latest" |
@@ -13,2 +13,8 @@ # longjohn | ||
## Production Use | ||
Longjohn collects a large amount of data in order to provide useful stack traces. While it is very helpful in | ||
development and testing environments, it is not recommended to use longjohn in production. The data collection puts | ||
a lot of strain on V8's garbage collector and can greatly slow down heavily-loaded applications. | ||
## Installation | ||
@@ -27,3 +33,5 @@ | ||
```javascript | ||
require('longjohn'); | ||
if (process.env.NODE_ENV !== 'production'){ | ||
require('longjohn'); | ||
} | ||
@@ -30,0 +38,0 @@ // ... your code |
Sorry, the diff of this file is not supported yet
Environment variable access
Supply chain riskPackage accesses environment variables, which may be a sign of credential stuffing or data theft.
Found 1 instance in 1 package
23665
302
53
1