turn-admin-client - npm Package Compare versions

Comparing version 0.1.3 to 0.1.4

lib/connection.js

@@ -7,3 +7,3 @@ 'use strict';
 
-  var options = options || {};
+  options = options || {};
 
@@ -24,4 +24,4 @@ var connection = net.createConnection(options.port, options.host);
   }
+
   return connection;
-
 };

lib/parser.js

 'use strict';
 
-var ipRegexString = '(?:[0-9]{1,3}\.){3}[0-9]{1,3}';
-var ipAddressRegexString = ipRegexString + ':\d+';
-var offOnRegexString = 'OFF|ON';
-var protocolRegexString = 'UDP|TCP';
-var timeRegexString = 'secs|mins|hours|days';
-
-var regex = /(\d)\)\s+id=(\d+), user\n<([a-zA-Z0-9\-\=\.]+)>:\n\s+started\s(\d+)\s(secs|mins|hours|days)\sago\n\s+expiring\sin\s(\d+)\s(secs|mins|hours|days)\n\s+client\sprotocol\s(UDP|TCP),\srelay\sprotocol\s(UDP|TCP)\n\s+client\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+),\sserver\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+)\n\s+relay\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+)\n\s+fingerprints\senforced:\s(OFF|ON)\n\s+mobile:\s(OFF|ON)\n\s+SHA256:\s(OFF|ON)\n\s+SHA\stype:\s(SHA1)\s+usage:\srp=(\d+),\srb=(\d+),\ssp=(\d+),\ssb=(\d+)\n\s+rate:\sr=(\d+),\ss=(\d),\stotal=(\d+)\s\(bytes\sper\ssec\)\n\s+peers:\n\s+([\s\.\d\n]+)$/;
-
+var ipRegexString = '(?:[0-9]{1,3}\\.){3}[0-9]{1,3}';
+var regex = /(\d)\)\s+id=(\d+), user <([a-zA-Z0-9\-\=\.]+)>:\n\s+started\s(\d+)\s(secs|mins|hours|days)\sago\n\s+expiring\sin\s(\d+)\s(secs|mins|hours|days)\n\s+client\sprotocol\s(UDP|TCP),\srelay\sprotocol\s(UDP|TCP)\n\s+client\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+),\sserver\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+)\n\s+relay\saddr\s((?:[0-9]{1,3}\.){3}[0-9]{1,3}:\d+)\n\s+fingerprints\senforced:\s(OFF|ON)\n\s+mobile:\s(OFF|ON)\n\s+SHA256:\s(OFF|ON)\n\s+SHA\stype:\s(SHA1)\s+usage:\srp=(\d+),\srb=(\d+),\ssp=(\d+),\ssb=(\d+)\n\s+rate:\sr=(\d+),\ss=(\d),\stotal=(\d+)\s\(bytes\sper\ssec\)\n\s+peers:\n\s+([\s\.\d\n]+)$/;
 var ipAddressRegex = new RegExp(ipRegexString);
-
 var captureKeys = [
@@ -57,3 +50,3 @@ 'recordNumber',
 
-      var ipAddresses = parsed[index].split("\n");
+      var ipAddresses = parsed[index].split('\n');
       ipAddresses.forEach(function getIpAddresses(line) {
@@ -60,0 +53,0 @@

lib/TurnAdminClient.js

@@ -32,3 +32,3 @@ 'use strict';
 
-  this.conn.write('ps');
+  this.conn.write('ps\n');
 
@@ -51,5 +51,4 @@ var sessions = [];
 
-  }, this.encoding, "\n\n");
+  }, this.encoding, '\n\n');
 
-
 };
@@ -56,0 +55,0 @@

package.json

 {
   "name": "turn-admin-client",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "A tool to access live TURN administrative information",
@@ -8,2 +8,6 @@ "keywords": [
   ],
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/danjenkins/node-turn-admin-client.git"
+  },
   "bugs": {
@@ -15,21 +19,23 @@ "url": "https://github.com/danjenkins/node-turn-admin-client/issues",
   "scripts": {
-    "test": "mocha spec/**/*.spec.js"
+    "test": "mocha spec/**/*.spec.js",
+    "jshint": "jshint ."
   },
-  "author": "Dan Jenkins <[e-mail]>",
+  "precommit": [
+    "jshint",
+    "test"
+  ],
   "license": "MIT",
+  "dependencies": {
+    "carrier": "^0.1.14"
+  },
   "devDependencies": {
+    "chai": "^1.9.1",
+    "jshint": "^2.5.6",
     "mocha": "^1.20.1",
-    "chai": "^1.9.1"
+    "precommit-hook": "^1.0.7"
   },
-  "dependencies": {
-    "carrier": "^0.1.14"
-  },
   "engines": {
     "node": ">=0.10.0 <0.12"
   },
-  "engineStrict": "true",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/danjenkins/node-turn-admin-client.git"
-  }
+  "engineStrict": "true"
 }

spec/api/api.spec.js

@@ -24,2 +24,4 @@ 'use strict';
       client.disconnect();
+      should.exist(sessions);
+      sessions.should.be.an('array');
 
@@ -26,0 +28,0 @@ //now check the sessions

spec/lib/testData.txt

@@ -1,3 +0,2 @@
-1) id=000000000000034864, user
-<1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29uMg==-ODk0NkFFMzktMTA2Qi00OTZELThCRDEtRDE2OTQzMkZGRjlF.1401812136>:
+1) id=000000000000034864, user <1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29uMg==-ODk0NkFFMzktMTA2Qi00OTZELThCRDEtRDE2OTQzMkZGRjlF.1401812136>:
       started 26 secs ago
@@ -20,4 +19,3 @@ expiring in 574 secs
 
-    2) id=000000000000034865, user
-<1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29u-RkQ2NkQ4OTEtMTNERC00MzJELTk3RjktMjU5OUI2QkEwNDM0.1401812145>:
+    2) id=000000000000034865, user <1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29u-RkQ2NkQ4OTEtMTNERC00MzJELTk3RjktMjU5OUI2QkEwNDM0.1401812145>:
       started 19 secs ago
@@ -24,0 +22,0 @@ expiring in 582 secs

spec/unit/lib/testString.txt

@@ -1,3 +0,2 @@
-1) id=000000000000034864, user
-<1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29uMg==-ODk0NkFFMzktMTA2Qi00OTZELThCRDEtRDE2OTQzMkZGRjlF.1401812136>:
+1) id=000000000000034864, user <1-QzZGODU5MDMtOTk3MS00QzQ0LThDMDQtNjA0ODk0ODUzMzVC-TWF0dEZyZWRyaWNrc29uMg==-ODk0NkFFMzktMTA2Qi00OTZELThCRDEtRDE2OTQzMkZGRjlF.1401812136>:
       started 26 secs ago
@@ -4,0 +3,0 @@ expiring in 574 secs

spec/unit/parser.spec.js

@@ -13,2 +13,3 @@ 'use strict';
     var results = parse(testString);
+    should.exist(results);
 
@@ -15,0 +16,0 @@ results.recordNumber.should.be.equal('1');

New alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

No contributors or author data

Maintenance

Package does not specify a list of contributors or an author in package.json.

Found 1 instance in 1 package

Fixed alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

15707

decreased by-30.31%

Dev dependency count

4

increased by100%

Lines of code

282

decreased by-0.35%

Number of low maintenance alerts

2

increased by100%

No dependency changes