Merge pull request #228 from web-push-libs/drop_node_0.10_0.12

Drop support for Node.js <= 0.12

Author: marco-c

.npmignore

@@ -2,5 +2,4 @@ coverage
 test
 test_tools
 .travis.yml
-wires
 .eslintrc

.travis.yml

@@ -15,8 +15,6 @@ env:
     - DISPLAY=$(if [[ $TRAVIS_OS_NAME == "linux" ]]; then echo ':99.0'; fi)
     - MOZ_XVFB=$(if [[ $TRAVIS_OS_NAME == "linux" ]]; then echo 1; fi)
   matrix:
-    - NODE_VERSION="0.10"
-    - NODE_VERSION="0.12"
     - NODE_VERSION="4"
     - NODE_VERSION="stable"
 before_install:
@@ -32,6 +30,5 @@ install:
 before_script:
   - if [[ $TRAVIS_OS_NAME == "linux" ]]; then sh -e /etc/init.d/xvfb start; fi
 script:
-  - if [[ $NODE_VERSION != "0.10" && $NODE_VERSION != "0.12" ]]; then $NPM run lint; fi
   - $NPM test
 sudo: required

package.json

@@ -9,7 +9,7 @@
   "scripts": {
     "download-browser": "node --harmony ./test/helpers/download-test-browsers.js",
     "lint": "node ./node_modules/eslint/bin/eslint --ignore-path .gitignore '.'",
-    "pretest": "npm run download-browser",
+    "pretest": "npm run lint && npm run download-browser",
     "test": "node --harmony node_modules/.bin/istanbul cover node_modules/.bin/_mocha -- --ui tdd test/test*"
   },
   "repository": {
@@ -29,12 +29,7 @@
   },
   "homepage": "https://github.com/marco-c/web-push#readme",
   "dependencies": {
-    "array.prototype.find": "^2.0.0",
     "asn1.js": "^4.8.1",
-    "bluebird": "^3.3.5",
-    "buffer-compare-shim": "^1.0.0",
-    "buffer-equals-polyfill": "^1.0.0",
-    "create-ecdh": "^4.0.0",
     "http_ece": "^0.5.1",
     "jws": "^3.1.3",
     "minimist": "^1.2.0",
@@ -62,6 +57,6 @@
     "which": "^1.2.11"
   },
   "engines": {
-    "node": ">= v0.10.0"
+    "node": ">= 4"
   }
 }

src/index.js

@@ -1,8 +1,5 @@
 'use strict';
 
-// This loads up shims required for older versions of node.
-require('./shim');
-
 const vapidHelper = require('./vapid-helper.js');
 const encryptionHelper = require('./encryption-helper.js');
 const WebPushLib = require('./web-push-lib.js');

src/shim.js

@@ -1,36 +1,26 @@
 'use strict';
 
-(function() {
-  const invalidNodeVersions = /0.(10|12).(\d+)/;
-  if (process.versions.node.match(invalidNodeVersions)) {
-    console.log('Skipping downloading browsers as selenium tests can\'t run on ' + process.versions.node);
-    return null;
-  }
+const seleniumAssistant = require('selenium-assistant');
 
-  /* eslint-disable global-require*/
-  const seleniumAssistant = require('selenium-assistant');
-  /* eslint-enable global-require*/
+let forceDownload = false;
+if (process.env.TRAVIS) {
+  forceDownload = true;
+}
 
-  let forceDownload = false;
-  if (process.env.TRAVIS) {
-    forceDownload = true;
-  }
+const promises = [
+  seleniumAssistant.downloadBrowser('firefox', 'stable', forceDownload),
+  seleniumAssistant.downloadBrowser('firefox', 'beta', forceDownload),
+  seleniumAssistant.downloadBrowser('firefox', 'unstable', forceDownload),
+  seleniumAssistant.downloadBrowser('chrome', 'stable', forceDownload),
+  seleniumAssistant.downloadBrowser('chrome', 'beta', forceDownload),
+  seleniumAssistant.downloadBrowser('chrome', 'unstable', forceDownload)
+];
 
-  const promises = [
-    seleniumAssistant.downloadBrowser('firefox', 'stable', forceDownload),
-    seleniumAssistant.downloadBrowser('firefox', 'beta', forceDownload),
-    seleniumAssistant.downloadBrowser('firefox', 'unstable', forceDownload),
-    seleniumAssistant.downloadBrowser('chrome', 'stable', forceDownload),
-    seleniumAssistant.downloadBrowser('chrome', 'beta', forceDownload),
-    seleniumAssistant.downloadBrowser('chrome', 'unstable', forceDownload)
-  ];
-
-  return Promise.all(promises)
-  .then(function() {
-    console.log('Download complete.');
-  })
-  .catch(function(err) {
-    console.error('Unable to download browsers.', err);
-    process.exit(1);
-  });
-})();
+Promise.all(promises)
+.then(function() {
+  console.log('Download complete.');
+})
+.catch(function(err) {
+  console.error('Unable to download browsers.', err);
+  process.exit(1);
+});