Merge pull request #80 from nodesecurity/send_version

Send version

README.md

@@ -127,6 +127,9 @@ And finally run it
 codeclimate analyze --dev
 ```
 
+## Suggesting Changes to Advisories
+Should you come across data in an advisory that you feel is wrong or is a false positive please let us know at [email protected]. We endeavor to make this process better in the future, however this is the best place to resolve these issues at the present.
+
 ## Contact
 
 Node Security (+) is brought to you by [^lift security](https://liftsecurity.io).

lib/check.js

@@ -9,7 +9,15 @@ var Path = require('path');
 var Wreck = require('wreck');
 var PathIsAbsolute = require('path-is-absolute');
 
-var Conf = require('rc')('nsp', { api: { baseUrl: 'https://api.nodesecurity.io', json: true } }, []);
+var Conf = require('rc')('nsp', {
+  api: {
+    baseUrl: 'https://api.nodesecurity.io',
+    json: true,
+    headers: {
+      'X-NSP-VERSION': require('../package.json').version
+    }
+  }
+}, []);
 
 var internals = {};
 internals.findLines = function (shrinkwrap, module, version) {

test/unit.js

@@ -5,6 +5,7 @@ var Lab = require('lab');
 var Nock = require('nock');
 var Path = require('path');
 var Check = require('../lib/check.js');
+var Pkg = require('../package.json');
 
 var lab = exports.lab = Lab.script();
 var describe = lab.describe;
@@ -248,5 +249,22 @@ describe('check', function () {
     });
   });
 
+  it('Includes X-NSP-VERSION in headers', function (done) {
+
+    Nock('https://api.nodesecurity.io', {
+      reqheaders: {
+        'X-NSP-VERSION': Pkg.version
+      }
+    })
+      .post('/check')
+      .reply(200, Findings);
+
+    Check(workingOptions, function (err, results) {
+
+      expect(err).to.not.exist();
+      done();
+    });
+  });
+
 
 });