raven - npm Package Compare versions

Comparing version 2.4.2 to 2.5.0

History.md

 # Changelog
 
+## 2.5.0 - 2018-04-09
+
+- feat: log non-standard ports in breadcrumbs (#440)
+- feat: add flag for unhandled promise rejections (#446)
+- fix: Remove a redundant try-catch block (#445)
+- fix: Do not override context when capturing non-error exceptions (#444)
+- fix: Update stack-trace to handle spaces in paths (#437)
+- docs: Remove private DSNs from the docs (#447)
+- docs: Update Usage docs to include Domains in Promise support (#438)
+
 ## 2.4.2 - 2018-02-27
 
-fix: Dont throw exception when called captureException without config (#431)
-fix: Preserve context in for rejected promises (#428)
-ref: Log promise rejection reason alongside eventid (#434)
-ref: Use named function for middlewares (#429)
+- fix: Dont throw exception when called captureException without config (#431)
+- fix: Preserve context in for rejected promises (#428)
+- ref: Log promise rejection reason alongside eventid (#434)
+- ref: Use named function for middlewares (#429)
 
@@ -10,0 +20,0 @@ ## 2.4.1 - 2018-02-09

lib/client.js

@@ -124,4 +124,6 @@ 'use strict';
       global.process.on('unhandledRejection', function(reason, promise) {
-        var context = promise.domain && promise.domain.sentryContext;
-        self.captureException(reason, context || {}, function(sendErr, eventId) {
+        var context = (promise.domain && promise.domain.sentryContext) || {};
+        context.extra = context.extra || {};
+        context.extra.unhandledPromiseRejection = true;
+        self.captureException(reason, context, function(sendErr, eventId) {
           if (!sendErr) {
@@ -335,3 +337,4 @@ var reasonMessage = (reason && reason.message) || reason;
     } else {
-      kwargs = kwargs || {};
+      // Do not use reference, as we'll modify this object later on
+      kwargs = kwargs ? JSON.parse(stringify(kwargs)) : {};
     }
@@ -342,9 +345,3 @@
     if (this.stacktrace) {
-      var ex;
-      // Generate a "synthetic" stack trace
-      try {
-        throw new Error(message);
-      } catch (ex1) {
-        ex = ex1;
-      }
+      var ex = new Error(message);
 
@@ -354,3 +351,3 @@ utils.parseStack(
         function(frames) {
-          // We trim last frame, as it's our `throw new Error(message)` call itself, which is redundant
+          // We trim last frame, as it's our `new Error(message)` statement itself, which is redundant
           kwargs.stacktrace = {
@@ -374,3 +371,8 @@ frames: frames.slice(0, -1)
     } else {
-      kwargs = kwargs || {};
+      var req = kwargs && kwargs.req;
+      // Do not use reference, as we'll modify this object later on
+      kwargs = kwargs ? JSON.parse(stringify(kwargs)) : {};
+      // Preserve `req` so we can use some framework's middleware methods
+      // `req` is the only thing that we pass through in `errorHandler`, so we need just that
+      if (req) kwargs.req = req;
     }
@@ -383,14 +385,12 @@
         var keys = Object.keys(err).sort();
-        var hash = md5(keys);
         var message =
           'Non-Error exception captured with keys: ' +
           utils.serializeKeysForMessage(keys);
-        var serializedException = utils.serializeException(err);
+        kwargs = extend(kwargs, {
+          message: message,
+          fingerprint: [md5(keys)],
+          extra: kwargs.extra || {}
+        });
+        kwargs.extra.__serialized__ = utils.serializeException(err);
 
-        kwargs.message = message;
-        kwargs.fingerprint = [hash];
-        kwargs.extra = {
-          __serialized__: serializedException
-        };
-
         err = new Error(message);
@@ -397,0 +397,0 @@ } else {

lib/instrumentation/http.js

@@ -22,7 +22,12 @@ 'use strict';
     } else {
-      this.__ravenBreadcrumbUrl =
-        (options.protocol || '') +
-        '//' +
-        (options.hostname || options.host || '') +
-        (options.path || '/');
+      var protocol = options.protocol || '';
+      var hostname = options.hostname || options.host || '';
+      // Don't log standard :80 (http) and :443 (https) ports to reduce the noise
+      var port =
+        !options.port || options.port === 80 || options.port === 443
+          ? ''
+          : ':' + options.port;
+      var path = options.path || '/';
+
+      this.__ravenBreadcrumbUrl = protocol + '//' + hostname + port + path;
     }
@@ -29,0 +34,0 @@ };

package.json

@@ -12,3 +12,3 @@ {
   ],
-  "version": "2.4.2",
+  "version": "2.5.0",
   "repository": "git://github.com/getsentry/raven-node.git",
@@ -36,3 +36,3 @@ "license": "BSD-2-Clause",
     "md5": "^2.2.1",
-    "stack-trace": "0.0.9",
+    "stack-trace": "0.0.10",
     "timed-out": "4.0.1",
@@ -39,0 +39,0 @@ "uuid": "3.0.0"

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

68812

increased by1.63%

Lines of code

1411

increased by0.36%

Dependency changes

• + Addedstack-trace@0.0.10(transitive)

• - Removedstack-trace@0.0.9(transitive)

• Updatedstack-trace@0.0.10