config-monitor - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

config/default.json

 {
-  "MonitorMin": {
+  "Monitor": {
     "appName": "config-monitor"
   }
 }

config/external.js

@@ -6,3 +6,3 @@ // Configuration overrides when NODE_ENV=external
   // Overrides for the monitor-min package
-  MonitorMin: {
+  Monitor: {
 
@@ -16,3 +16,3 @@ // This setting allows incoming monitor connections from remote systems.
   // Overrides for the node-monitor application
-  Monitor: {
+  Dashboard: {
 
@@ -19,0 +19,0 @@ // This setting allows incoming browser connections from remote systems.

config/runtime.json

@@ -1,4 +0,1 @@
-{
-  "hello": "init",
-  "a": "callforit"
-}
+{}

History.md

@@ -1,10 +0,14 @@
-0.1.1 / 10-21-2012
+0.1.2 / 2013-11-06
 ==================
 
-  * Separate basic view from view with sources
-  * A better README.md
+  * Update to monitor 6.0
 
-0.1.0 / 10-20-2012
+0.1.1 / 2013-10-23
 ==================
 
+  * Exposed newTray as a probe control
+
+0.1.0 / 2013-10-20
+==================
+
   * Initial checkin

lib/index.js

@@ -1,10 +0,17 @@
-// This is run when the app is loaded from node-monitor
+// This is run when the app is loaded from monitor-dashboard
 (function(root){
 
   // Create a server, and expose this directory
-  var Connect = require('connect');
-  var Static = Connect['static'](__dirname);
+  var Connect = require('connect'),
+      FS = require('fs'),
+      Path = require('path'),
+      Static = Connect['static'](__dirname);
 
-  // Load server side probes here
-  require('./probe/ConfigProbe.js');
+  // Load all probes found in the ./probe directory
+  // This is synchronous because require() is synchronous
+  FS.readdirSync(Path.join(__dirname, 'probe')).forEach(function(fileName) {
+    if (fileName.substr(-3) === '.js') {
+      require('./probe/' + fileName);
+    }
+  });
 
@@ -11,0 +18,0 @@ // Export a middleware component

lib/probe/ConfigProbe.js

@@ -5,3 +5,3 @@ // ConfigProbe.js (c) 2013 Derrick West
 // http://derrickwest.github.com/config-monitor-min
-var Monitor = require('monitor-min'),
+var Monitor = require('monitor'),
     Probe = Monitor.Probe,
@@ -8,0 +8,0 @@ Config = require('config');

monitor.js

 // Bootstrap for standalone development
-require('monitor');
+require('monitor-dashboard');

package.json

 {
   "name": "config-monitor",
   "description": "Monitoring for the config module",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "main": "lib/index.js",
@@ -16,4 +16,4 @@ "author": "Derrick West <[e-mail]>",
     "connect": ">=1.7.2 <1.8.0",
-    "monitor": ">=0.5.2 <0.6.0",
-    "monitor-min": ">=0.5.8 <0.6.0"
+    "monitor": ">=0.6.0 <0.7.0",
+    "monitor-dashboard": ">=0.6.0 <0.7.0"
   },
@@ -20,0 +20,0 @@ "devDependencies": {

README.md

@@ -15,3 +15,3 @@ config-monitor
       ...
-      "monitor-min": ">=0.5.8 <0.6.0",
+      "monitor": ">=0.6.0 <0.7.0",
       "config-monitor": ">=0.1.1 <0.2.0",
@@ -23,17 +23,18 @@ ...
     // Start monitoring, and load monitor probes
-    var Monitor = require('monitor-min').start();
+    var Monitor = require('monitor').start();
     var configMonitor = require('config-monitor');
 
-Adding to Node Monitor
-----------------------
+Adding to the Monitor Dashboard
+-------------------------------
 
-To add these monitors to your node monitor UI, install config-monitor in your node path (or globally)
+To add these monitors to your dashboard, install config-monitor in your node path (or globally)
 
+    npm install -g monitor-dashboard
     npm install -g config-monitor
 
-Then start node monitor
+Then start the dashboard
 
-    npm start monitor
+    npm start monitor-dashboard
 
 When adding components into your dashboard, the config-monitor components will be avilable.
 

site_db/Page/index.json

@@ -17,3 +17,6 @@ {
       "monitor": {
-        "probeClass": "Config"
+        "probeClass": "Config",
+        "hostName": "Derrick.local",
+        "appName": "config-monitor",
+        "appInstance": "1"
       }
@@ -20,0 +23,0 @@ }

test/ConfigProbeTest.js

 // ConfigProbeTest.js (c) 2013 Derrick West and other contributors
 // May be freely distributed under the MIT license.
 // For further details and documentation:
-// http://derrickwest.github.com/config-monitor-min
+// http://derrickwest.github.com/config-monitor
 (function(root){
 
   // Dependencies
-  var Monitor = require('monitor-min'),
+  var Monitor = require('monitor'),
+      configMonitor = require('../lib/index'),
       CONFIG = require('config'),
-      ConfigProbe = Monitor.ConfigProbe,
       Backbone = Monitor.Backbone, _ = Monitor._;
@@ -35,12 +35,2 @@
     /**
-    * Tests that classes are in correct
-    * @method Config-Classes
-    */
-    Classes: function(test) {
-      test.ok(ConfigProbe.prototype instanceof Backbone.Model, 'The data model is in place');
-      test.ok(ConfigProbe.prototype instanceof Monitor.Probe, 'It is a probe');
-      test.done();
-    },
-
-    /**
     * Tests the initial config values
@@ -52,8 +42,8 @@ * @method Config-InitialValues
       configMonitor.connect(function() {
-        var json = configMonitor.toJSON();
-        test.ok(json.MonitorMin != null, 'Monitor configuration is present');
-        test.equal(json.MonitorMin.appName, 'MonitorMin', 'The appName parameter has the correct value');
-        test.equal(json.MonitorMin.serviceBasePort, 42000, 'The serviceBasePort parameter has the correct value');
-        test.equal(json.MonitorMin.portsToScan, 20, 'The portsToScan parameter has the correct value');
-        test.equal(json.MonitorMin.allowExternalConnections, false, 'The allowExternalConnections parameter has the correct value');
+        var json = configMonitor.toJSON().config;
+        test.ok(json.Monitor != null, 'Monitor configuration is present');
+        test.equal(json.Monitor.appName, 'config-monitor', 'The appName parameter has the correct value');
+        test.equal(json.Monitor.serviceBasePort, 42000, 'The serviceBasePort parameter has the correct value');
+        test.equal(json.Monitor.portsToScan, 20, 'The portsToScan parameter has the correct value');
+        test.equal(json.Monitor.allowExternalConnections, false, 'The allowExternalConnections parameter has the correct value');
         test.done();
@@ -60,0 +50,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

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

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

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

Improved metrics

Total package byte prevSize

28367

increased by0.38%

Number of lines in readme file

39

increased by2.63%

Number of medium supply chain risk alerts

0

decreased by-100%

Worsened metrics

Lines of code

359

decreased by-0.28%

Number of low supply chain risk alerts

2

increased byInfinity%

Dependency changes

• + Addedmonitor-dashboard@>=0.6.0 <0.7.0

• + Addedcore-monitor@0.6.3(transitive)

• + Addedmonitor@0.6.10(transitive)

• + Addedmonitor-dashboard@0.6.7(transitive)

• - Removedmonitor-min@>=0.5.8 <0.6.0

• - Removedcore-monitor@0.5.4(transitive)

• - Removedmonitor@0.5.4(transitive)

• - Removedmonitor-min@0.5.13(transitive)

• Updatedmonitor@>=0.6.0 <0.7.0