restify - npm Package Compare versions

Comparing version 2.3.2 to 2.3.3

CHANGES.md

 # restify Changelog
 
-## 2.3.3 (not yet released)
+## 2.3.4 (not yet released)
 
+## 2.3.3
+
+- Stop logging client_req in bunyan output
+- GH-319 make DTrace optional
+- GH-335 Content-Type'd routes not accepting array (Pedro Palazón)
+
 ## 2.3.2
@@ -6,0 +12,0 @@

lib/bunyan_helper.js

@@ -123,2 +123,5 @@ // Copyright 2012 Mark Cavage, Inc.  All rights reserved.
 function clientReq(req) {
+        if (!req)
+                return (req);
+
         var host;
@@ -153,3 +156,11 @@
 
+var SERIALIZERS = {
+        err: bunyan.stdSerializers.err,
+        req: bunyan.stdSerializers.req,
+        res: bunyan.stdSerializers.res,
+        client_req: clientReq,
+        client_res: clientRes
+};
 
+
 ///--- Exports
@@ -159,9 +170,3 @@
         RequestCaptureStream: RequestCaptureStream,
-        serializers: {
-                err: bunyan.stdSerializers.err,
-                req: bunyan.stdSerializers.req,
-                res: bunyan.stdSerializers.res,
-                client_req: clientReq,
-                client_res: clientRes
-        },
+        serializers: SERIALIZERS,
 
@@ -171,3 +176,3 @@ createLogger: function createLogger(name) {
                         name: name,
-                        serializers: module.exports.serializers,
+                        serializers: SERIALIZERS,
                         streams: [ {
@@ -174,0 +179,0 @@ level: 'warn',

lib/dtrace.js

 // Copyright 2012 Mark Cavage, Inc.  All rights reserved.
 
-var dtrace = require('dtrace-provider');
 
 
-
 ///--- Globals
 
-var DTraceProvider = dtrace.DTraceProvider;
-
 var ID = 0;
@@ -44,3 +40,19 @@ var MAX_INT = Math.pow(2, 32) - 1;
         if (!PROVIDER) {
-                PROVIDER = dtrace.createDTraceProvider('restify');
+                try {
+                        var dtrace = require('dtrace-provider');
+                        PROVIDER = dtrace.createDTraceProvider('restify');
+                } catch (e) {
+                        PROVIDER = {
+                                fire: function () {},
+                                enable: function () {},
+                                addProbe: function () {
+                                        var p = {
+                                                fire: function () {}
+                                        };
+                                        return (p);
+                                },
+                                removeProbe: function () {},
+                                disable: function () {}
+                        };
+                }
 
@@ -47,0 +59,0 @@ PROVIDER._rstfy_probes = {};

lib/router.js

@@ -174,3 +174,2 @@ // Copyright 2012 Mark Cavage, Inc.  All rights reserved.
         assert.string(options.name, 'options.name');
-        assert.optionalString(options.contentType, 'options.contentType');
 
@@ -177,0 +176,0 @@ var exists;

package.json

@@ -28,3 +28,3 @@ {
         "description": "REST framework",
-        "version": "2.3.2",
+        "version": "2.3.3",
         "repository": {
@@ -49,3 +49,2 @@ "type": "git",
                 "deep-equal": "0.0.0",
-                "dtrace-provider": "0.2.8",
                 "formidable": "1.0.11",
@@ -64,2 +63,5 @@ "http-signature": "0.9.11",
         },
+        "optionalDependencies": {
+                "dtrace-provider": "0.2.8"
+        },
         "devDependencies": {
@@ -66,0 +68,0 @@ "cover": "0.2.8",

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

177797

increased by0.21%

Worsened metrics

Number of package files

49

decreased by-2%

Lines of code

3994

decreased by-0.03%

Dependency changes

• - Removeddtrace-provider@0.2.8