node-ral - npm Package Compare versions

Comparing version 0.6.2 to 0.7.0

lib/ext/converter/jsonConverter.js

@@ -39,2 +39,6 @@ /**
         buffer = iconv.encode(JSON.stringify(data), config.encoding);
+        if (!config.skipContentLength) {
+            config.headers = config.headers || {};
+            config.headers['Content-Length'] = buffer.length;
+        }
     }
@@ -41,0 +45,0 @@ catch (ex) {

lib/ext/converter/rawConverter.js

@@ -26,2 +26,4 @@ /**
 RawConverter.prototype.pack = function (config, data) {
+    config.headers = config.headers || {};
+    config.headers['Content-Length'] = data.length;
     return data;
@@ -28,0 +30,0 @@ };

lib/ext/converter/stringConverter.js

@@ -40,2 +40,6 @@ /**
         buffer = iconv.encode(data.toString(), config.encoding);
+        if (!config.skipContentLength) {
+            config.headers = config.headers || {};
+            config.headers['Content-Length'] = buffer.length;
+        }
     }
@@ -42,0 +46,0 @@ catch (e) {

lib/ext/protocol/httpProtocolBase.js

@@ -20,3 +20,5 @@ /**
 var urlParse = require('url').parse;
+var HttpsProxyAgent = require('https-proxy-agent');
 
+
 function HttpProtocolBase() {
@@ -157,6 +159,12 @@ Protocol.call(this);
         }
-        opt.path = url.resolve(uri, path);
-        var proxyUri = url.parse(proxy);
-        opt.host = proxyUri.hostname;
-        opt.port = proxyUri.port ? proxyUri.port : (proxyUri.protocol === 'http:' ? 80 : 433);
+        if (opt.protocol === 'https:') {
+            var agent = new HttpsProxyAgent(proxy);
+            opt.agent = agent;
+        }
+        else {
+            opt.path = url.resolve(uri, path);
+            var proxyUri = url.parse(proxy);
+            opt.host = proxyUri.hostname;
+            opt.port = proxyUri.port ? proxyUri.port : (proxyUri.protocol === 'http:' ? 80 : 433);
+        }
     }
@@ -163,0 +171,0 @@

package.json

 {
   "name": "node-ral",
-  "version": "0.6.2",
+  "version": "0.7.0",
   "description": "a rpc client for node",
@@ -30,2 +30,3 @@ "main": "index.js",
     "form-data": "0.1.4",
+    "https-proxy-agent": "1.0.0",
     "iconv-lite": "0.4.13",
@@ -32,0 +33,0 @@ "performance-now": "0.2.0",

test/converter.js

@@ -88,5 +88,7 @@ /**
         };
-        var pack = jsonConverter.pack(mockUTF8Context, data);
-        var unpack = jsonConverter.unpack(mockUTF8Context, pack);
+        var options = _.clone(mockUTF8Context);
+        var pack = jsonConverter.pack(options, data);
+        var unpack = jsonConverter.unpack(_.clone(mockUTF8Context), pack);
         data.should.be.eql(unpack);
+        options.headers['Content-Length'].should.be.eql(20);
     });
@@ -98,5 +100,5 @@
         var data = '张三李四';
-        var pack = converter.pack(mockUTF8Context, data);
+        var pack = converter.pack(_.clone(mockUTF8Context), data);
         (function () {
-            jsonConverter.unpack(mockUTF8Context, pack);
+            jsonConverter.unpack(_.clone(mockUTF8Context), pack);
         }).should.throwError();
@@ -111,7 +113,23 @@ });
         };
-        var pack = jsonConverter.pack(mockGBKContext, data);
-        var unpack = jsonConverter.unpack(mockGBKContext, pack);
+        var options = _.clone(mockGBKContext);
+        var pack = jsonConverter.pack(options, data);
+        var unpack = jsonConverter.unpack(_.clone(mockGBKContext), pack);
         data.should.be.eql(unpack);
+        options.headers['Content-Length'].should.be.eql(18);
     });
 
+    it('pack and unpack could skip header', function () {
+        var jsonConverter = new JsonConverter();
+        var data = {
+            a: 1,
+            b: '张三'
+        };
+        var options = _.clone(mockGBKContext);
+        options.skipContentLength = true;
+        var pack = jsonConverter.pack(options, data);
+        var unpack = jsonConverter.unpack(_.clone(mockGBKContext), pack);
+        data.should.be.eql(unpack);
+        (options.headers === undefined).should.be.true;
+    });
+
     it('pack should fail if encoding is illegal', function () {
@@ -121,3 +139,3 @@ var converter = new JsonConverter();
         (function () {
-            converter.pack(mockBlahContext, data);
+            converter.pack(_.clone(mockBlahContext), data);
         }).should.throwError();
@@ -130,3 +148,3 @@ });
         (function () {
-            jsonConverter.pack(mockUTF8Context, data);
+            jsonConverter.pack(_.clone(mockUTF8Context), data);
         }).should.not.throwError();
@@ -150,5 +168,7 @@ });
         var data = '张三李四';
-        var pack = converter.pack(mockUTF8Context, data);
-        var unpack = converter.unpack(mockUTF8Context, pack);
+        var options = _.clone(mockUTF8Context);
+        var pack = converter.pack(options, data);
+        var unpack = converter.unpack(_.clone(mockUTF8Context), pack);
         data.should.be.eql(unpack);
+        options.headers['Content-Length'].should.be.eql(12);
     });
@@ -159,7 +179,20 @@
         var data = '张三李四';
-        var pack = converter.pack(mockGBKContext, data);
-        var unpack = converter.unpack(mockGBKContext, pack);
+        var options = _.clone(mockGBKContext);
+        var pack = converter.pack(options, data);
+        var unpack = converter.unpack(_.clone(mockGBKContext), pack);
         data.should.be.eql(unpack);
+        options.headers['Content-Length'].should.be.eql(8);
     });
 
+    it('pack and unpack could skip header', function () {
+        var converter = new StringConverter();
+        var data = '张三李四';
+        var options = _.clone(mockGBKContext);
+        options.skipContentLength = true;
+        var pack = converter.pack(options, data);
+        var unpack = converter.unpack(_.clone(mockGBKContext), pack);
+        data.should.be.eql(unpack);
+        (options.headers === undefined).should.be.true;
+    });
+
     it('pack should fail if encoding is illegal', function () {
@@ -169,3 +202,3 @@ var converter = new StringConverter();
         (function () {
-            converter.pack(mockBlahContext, data);
+            converter.pack(_.clone(mockBlahContext), data);
         }).should.throwError();
@@ -178,3 +211,3 @@ });
         (function () {
-            converter.pack(mockUTF8Context, data);
+            converter.pack(_.clone(mockUTF8Context), data);
         }).should.not.throwError();
@@ -181,0 +214,0 @@ });

test/protocol/http_protocol_post_test.js

@@ -144,2 +144,3 @@ /**
         if (pathname === '/hello' && request.method === 'POST') {
+
             var formIn = new formidable.IncomingForm();
@@ -159,2 +160,3 @@ // var decodeStream = iconv.decodeStream(encoding || 'utf-8');
             formIn.parse(request, function (err, fields, files) {
+
                 var content = 'hear you';
@@ -161,0 +163,0 @@ response.writeHead(200, {

test/ral.js

@@ -692,3 +692,3 @@ /**
         }).on('data', function (data) {
-            data.should.match(/STATUS OK/);
+            data.should.match(/html/);
             done();
@@ -712,3 +712,3 @@ }).on('error', function (err) {
         }).on('data', function (data) {
-            data.should.match(/STATUS OK/);
+            data.should.match(/html/);
             done();
@@ -715,0 +715,0 @@ }).on('error', function (err) {

New alerts

Network access

Supply chain risk

This module accesses the network.

Found 1 instance in 1 package

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

332817

increased by1.45%

Number of package files

91

increased by1.11%

Lines of code

9728

increased by1.33%

Worsened metrics

Dependency count

9

increased by12.5%

Number of low supply chain risk alerts

29

increased by7.41%

Number of medium supply chain risk alerts

12

increased by9.09%

Dependency changes

• + Addedhttps-proxy-agent@1.0.0

• + Addedagent-base@2.1.1(transitive)

• + Addeddebug@2.6.9(transitive)

• + Addedhttps-proxy-agent@1.0.0(transitive)

• + Addedms@2.0.0(transitive)

• + Addedsemver@5.0.3(transitive)