ESLint fixes: const, newlines, destructuring

.eslintrc.json

@@ -18,6 +18,10 @@
 			"error",
 			"always"
 		],
-		"prefer-const": "error"
+		"prefer-const": "error",
+		"prefer-destructuring": "error",
+		"no-var": "error",
+		"strict": "error",
+		"eol-last": "error"
 	}
 }

actions/createProxyServer.js

@@ -1,12 +1,14 @@
-var fs = require('fs-extra');
-var shell = require('shelljs');
+'use strict';
 
-var npath = require('../utils/nginxPath');
-var conf = require('../utils/nginxConf');
-var nginxReload = require('../utils/nginxReload');
-var appendToList = require('../utils/listFile').appendToList;
+const fs = require('fs-extra');
+const shell = require('shelljs');
 
-var EOL = require('os').EOL; // \n if used on Linux, \r\n if used on Windows.
+const npath = require('../utils/nginxPath');
+const conf = require('../utils/nginxConf');
+const nginxReload = require('../utils/nginxReload');
+const { appendToList } = require('../utils/listFile');
+
+const { EOL } = require('os'); // \n if used on Linux, \r\n if used on Windows.
 
 function createProxyServer(domain, inPort, outPort) {
 	outPort = outPort || 80;
@@ -37,4 +39,4 @@ function createProxyServer(domain, inPort, outPort) {
 	nginxReload();
 }
 
-module.exports = createProxyServer;
+module.exports = createProxyServer;

actions/createStaticServer.js

@@ -1,14 +1,16 @@
-var fs = require('fs-extra');
-var shell = require('shelljs');
-var path = require('path');
+'use strict';
 
-var npath = require('../utils/nginxPath');
-var conf = require('../utils/nginxConf');
-var nginxReload = require('../utils/nginxReload');
-var appendToList = require('../utils/listFile').appendToList;
+const fs = require('fs-extra');
+const shell = require('shelljs');
+const path = require('path');
 
-var currentPath = path.normalize(process.cwd());
-var EOL = require('os').EOL; // \n if used on Linux, \r\n if used on Windows.
+const npath = require('../utils/nginxPath');
+const conf = require('../utils/nginxConf');
+const nginxReload = require('../utils/nginxReload');
+const { appendToList } = require('../utils/listFile');
+
+const currentPath = path.normalize(process.cwd());
+const { EOL } = require('os'); // \n if used on Linux, \r\n if used on Windows.
 
 function createStaticServer(domain, outPort) {
 	outPort = outPort || 80;
@@ -38,4 +40,4 @@ function createStaticServer(domain, outPort) {
 	nginxReload();
 }
 
-module.exports = createStaticServer;
+module.exports = createStaticServer;

actions/killALL.js

@@ -1,7 +1,9 @@
-var shell = require('shelljs');
+'use strict';
 
-var npath = require('../utils/nginxPath');
-var conf = require('../utils/nginxConf');
+const shell = require('shelljs');
+
+const npath = require('../utils/nginxPath');
+const conf = require('../utils/nginxConf');
 
 function killALL () {
 	shell.rm('-Rf', npath.serversBakUp);
@@ -17,4 +19,4 @@ function killALL () {
 	shell.ln('-sf', npath.confD() + "default.conf", npath.enabledSites() + "default.conf"); // Symlink the default.conf file from confD to sites-enabled	
 }
 
-module.exports = killALL;
+module.exports = killALL;

actions/killAllConfirm.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const prompt = require('prompt');
 const shell = require('shelljs');
 
@@ -8,7 +10,7 @@ function killAllConfirm () {
 
 	prompt.start();
 
-	var property = {
+	const property = {
 		name: 'yesno',
 		message: 'This will completely destroy all configs and reset nginx. Are you sure?',
 		validator: /y[es]*|n[o]?/,
@@ -29,4 +31,4 @@ function killAllConfirm () {
 	});
 }
 
-module.exports = killAllConfirm;
+module.exports = killAllConfirm;

actions/killServer.js

@@ -1,9 +1,11 @@
-var shell = require('shelljs');
+'use strict';
 
-var npath = require('../utils/nginxPath');
-var conf = require('../utils/nginxConf');
-var nginxReload = require('../utils/nginxReload');
-var removeFromList = require('../utils/listFile').removeFromList;
+const shell = require('shelljs');
+
+const npath = require('../utils/nginxPath');
+const conf = require('../utils/nginxConf');
+const nginxReload = require('../utils/nginxReload');
+const { removeFromList } = require('../utils/listFile');
 
 function killServer(domain, outPort) {
 	shell.rm('-rf', conf(npath.enabledSites(), domain, outPort));
@@ -14,4 +16,4 @@ function killServer(domain, outPort) {
 	nginxReload();
 }
 
-module.exports = killServer;
+module.exports = killServer;

actions/listServers.js

@@ -1,12 +1,14 @@
-var readServers = require('../utils/listFile').readServers;
-var prettyjson = require('prettyjson');
+'use strict';
 
-var EOL = require('os').EOL;
+const { readServers } = require('../utils/listFile');
+const prettyjson = require('prettyjson');
+
+const { EOL } = require('os');
 
 function listServers() {
-	var serversList = readServers();
+	const serversList = readServers();
 	if(serversList) console.log(EOL + prettyjson.render(serversList) + EOL);
 	else console.log("\nNo servers were found! Create some using `up`!\n");
 }
 
-module.exports = listServers;
+module.exports = listServers;

build/fetchTLDS.js

@@ -1,7 +1,9 @@
-var https = require('https');
-var fs = require('fs-extra');
+'use strict';
 
-var file = fs.createWriteStream("./assets/tlds.txt");
+const https = require('https');
+const fs = require('fs-extra');
+
+const file = fs.createWriteStream("./assets/tlds.txt");
 https.get("https://data.iana.org/TLD/tlds-alpha-by-domain.txt", function(response) {
 	response.pipe(file);
-});
+});

index.js

@@ -3,20 +3,20 @@
 'use strict';
 
 // Requiring npm modules
-var program = require('commander');
-var chalk = require('chalk');
+const program = require('commander');
+const chalk = require('chalk');
 //var fs = require('fs-extra');
 
 // Requiring Actions
-var createProxyServer = require('./actions/createProxyServer');
-var createStaticServer = require('./actions/createStaticServer');
-var killServer = require('./actions/killServer');
-var listServers = require('./actions/listServers');
-var killAllConfirm = require('./actions/killAllConfirm');
+const createProxyServer = require('./actions/createProxyServer');
+const createStaticServer = require('./actions/createStaticServer');
+const killServer = require('./actions/killServer');
+const listServers = require('./actions/listServers');
+const killAllConfirm = require('./actions/killAllConfirm');
 
 // Requiring utils
-var validate = require('./utils/validate');
-var requirements = require('./utils/requirements');
+const validate = require('./utils/validate');
+const requirements = require('./utils/requirements');
 
 // Check for requirements such as OS version and nginx install. Throw and exit if requirements not found.
 // #Roadmap: Add ability to satisfy any possible requirements.
@@ -93,4 +93,4 @@ program.on('--help', function () {
 });
 
 // Parses commands passed to `up` and chooses one of the above commands.
-program.parse(process.argv);
+program.parse(process.argv);

utils/isFQDN.js

@@ -3,7 +3,7 @@
 
 'use strict';
 
-var fs = require('fs-extra');
+const fs = require('fs-extra');
 
 // Official list of TLDs should be fetched from:
 // https://data.iana.org/TLD/tlds-alpha-by-domain.txt
@@ -14,7 +14,7 @@ var fs = require('fs-extra');
 function isFQDN(domain) {
 
 	// Importing and parsing `tlds.txt` file
-	var tlds = fs.readFileSync('./assets/tlds.txt', 'utf8')
+	const tlds = fs.readFileSync('./assets/tlds.txt', 'utf8')
 		.split(/[\r\n]+/)
 		.filter(x => !x.startsWith('#'));
 		
@@ -22,19 +22,19 @@ function isFQDN(domain) {
 		return false;
 	}
 
-	var labels = domain.split('.').reverse();
+	const labels = domain.split('.').reverse();
 
 	if (labels.length < 2) {
 		return false;
 	}
 
-	var tld = labels[0];
+	const [ tld ] = labels;
 
 	if (!tlds.includes(tld.toUpperCase())) {
 		return false;
 	}
 
-	for (var label of labels) {
+	for (const label of labels) {
 
 		const len = label.length;
 
@@ -56,6 +56,6 @@ function isFQDN(domain) {
 		}
 	}
 	return true;
-};
+}
 
-module.exports = isFQDN;
+module.exports = isFQDN;

utils/isIP.js

@@ -1,14 +1,16 @@
+'use strict';
+
 // Parses a string, and returns true if it is an IP Address.
 
 function isIP(str) {
-	var segments = str
+	const segments = str
 		.split(".")
 		.map(Number);
 	if (!segments.length === 4) {
 		return false;
 	}
-	for(var i = 0; i < segments.length; i++) {
-		var segment = segments[i];
+	for(let i = 0; i < segments.length; i++) {
+		const segment = segments[i];
 		if (Number.isNaN(segment)) {
 			return false;
 		}
@@ -22,4 +24,4 @@ function isIP(str) {
 	return true;
 }
 
-module.exports = isIP;
+module.exports = isIP;

utils/listFile.js

@@ -1,13 +1,15 @@
-var fs = require('fs-extra');
+'use strict';
 
-var removeFromArray = require('./removeFromArray');
-var listFilePath = require('./nginxPath').serversUp;
+const fs = require('fs-extra');
+
+const removeFromArray = require('./removeFromArray');
+const listFilePath = require('./nginxPath').serversUp;
 
 function appendToList(domain, outPort, inPort) {
 	
 	inPort = inPort || undefined;
-	var jsonFile = { "domains": [] };
-	var domBlock = {
+	let jsonFile = { "domains": [] };
+	const domBlock = {
 		"domain": domain,
 		"outPort": outPort
 	};
@@ -20,7 +22,7 @@ function appendToList(domain, outPort, inPort) {
 	}
 
 	if (fs.existsSync(listFilePath())) {
-		var jsonBuffer = JSON.parse(fs.readFileSync(listFilePath()));
+		const jsonBuffer = JSON.parse(fs.readFileSync(listFilePath()));
 		jsonFile.domains = removeFromArray(jsonBuffer.domains, domain, outPort);
 	}
 	jsonFile.domains.push(domBlock);
@@ -30,8 +32,8 @@ function appendToList(domain, outPort, inPort) {
 
 function removeFromList (domain, outPort) {
 	if (fs.existsSync(listFilePath())) {
-		var jsonFile = { "domains": [] };
-		var jsonBuffer = JSON.parse(fs.readFileSync(listFilePath()));
+		let jsonFile = { "domains": [] };
+		const jsonBuffer = JSON.parse(fs.readFileSync(listFilePath()));
 		jsonFile.domains = removeFromArray(jsonBuffer.domains, domain, outPort);
 
 		jsonFile = JSON.stringify(jsonBuffer, null, '\t');
@@ -41,7 +43,7 @@ function removeFromList (domain, outPort) {
 }
 
 function readServers () {
-	var serversList = JSON.parse(fs.readFileSync(listFilePath()));
+	const serversList = JSON.parse(fs.readFileSync(listFilePath()));
 
 	if(!serversList.domains[0]) return undefined;
 	return serversList;
@@ -49,4 +51,4 @@ function readServers () {
 
 module.exports.appendToList = appendToList;
 module.exports.readServers = readServers;
-module.exports.removeFromList = removeFromList;
+module.exports.removeFromList = removeFromList;

utils/nginxConf.js

@@ -1,7 +1,9 @@
+'use strict';
+
 // Simple function that takes a path and domain name, concatenates them with ".conf" and returns it.
 
 function conf(path, domain, outPort) {
 	return (path + domain + "." + outPort + ".conf");
 }
 
-module.exports = conf;
+module.exports = conf;

utils/nginxPath.js

@@ -2,12 +2,12 @@
 
 // These functions just return paths. Later, these should be modified to poll from nginx's config.
 
-var npath = "/etc/nginx/";
-var enabled = npath + "sites-enabled/";
-var confDpath = npath + "conf.d/";
-var upPath = "/etc/up-serve/";
-var wwwRoot = upPath + "static/";
-var serverListPath = upPath + "servers";
+const npath = "/etc/nginx/";
+const enabled = npath + "sites-enabled/";
+const confDpath = npath + "conf.d/";
+const upPath = "/etc/up-serve/";
+const wwwRoot = upPath + "static/";
+const serverListPath = upPath + "servers";
 
 function nginxPath() {
 	return npath;
@@ -46,4 +46,4 @@ module.exports.enabledSites = enabledSites;
 module.exports.webRoot = webRoot;
 module.exports.webRootDomain = webRootDomain;
 module.exports.serversUp = serversUp;
-module.exports.serversBakUp = serversBakUp;
+module.exports.serversBakUp = serversBakUp;

utils/nginxReload.js

@@ -1,4 +1,6 @@
-var execSync = require('child_process').execSync;
+'use strict';
+
+const { execSync } = require('child_process');
 
 function nginxReload() {
 	execSync('service nginx reload', function (error, stdout, stderr) {
@@ -11,4 +13,4 @@ function nginxReload() {
 	});
 }
 
-module.exports = nginxReload;
+module.exports = nginxReload;

utils/parseToInt.js

@@ -1,8 +1,10 @@
+'use strict';
+
 // Parse an input string and return a number if it is an integer. If it's a float, string, or array, return undefined.
 
 function parseToInt(inputString) {
-	var parsing = /^\d+$/.exec(inputString);
+	const parsing = /^\d+$/.exec(inputString);
 	return (parsing || [])[0];
 }
 
-module.exports = parseToInt;
+module.exports = parseToInt;

utils/removeFromArray.js

@@ -1,7 +1,11 @@
+'use strict';
+
 function removeFromArray (arr, dom, port) {
-	var shouldDelete = [];
+	let shouldDelete = [];
 
-	for(var i = 0; i < arr.length; i++) if((arr[i].domain == dom) && (arr[i].outPort == port)) shouldDelete = [true, i];
+	for(let i = 0; i < arr.length; i++)
+		if((arr[i].domain == dom) && (arr[i].outPort == port))
+			shouldDelete = [true, i];
 
 	if (shouldDelete[0]) {
 		arr.splice(shouldDelete[1], 1);
@@ -9,4 +13,4 @@ function removeFromArray (arr, dom, port) {
 	return arr;
 }
 
-module.exports = removeFromArray;
+module.exports = removeFromArray;

utils/requirements.js

@@ -1,10 +1,12 @@
-var shell = require('shelljs');
-var chalk = require('chalk');
+'use strict';
+
+const shell = require('shelljs');
+const chalk = require('chalk');
 
 function requirements() {
 
 	// Detect Linux or BSD
-	var isLin = /^linux|^bsd/.test(process.platform);
+	const isLin = /^linux|^bsd/.test(process.platform);
 	
 	// Throw if OS is not Linux or BSD. This should be changed to throw if not Debian based distro. Eventually, we can add more exceptions as `up` handles more cases.
 	if(!isLin) {
@@ -20,4 +22,4 @@ function requirements() {
 
 }
 
-module.exports = requirements;
+module.exports = requirements;

utils/validate.js

@@ -1,8 +1,10 @@
-var parseToInt = require('./parseToInt');
-var isIP = require('./isIP');
+'use strict';
+
+const parseToInt = require('./parseToInt');
+const isIP = require('./isIP');
 
 // Using Validator
-var isDomain = require('./isFQDN');
+const isDomain = require('./isFQDN');
 
 function validate(domain, inPort, outPort) {
 	//
@@ -10,15 +12,15 @@ function validate(domain, inPort, outPort) {
 	outPort = outPort || 80;
 
 	// Error messages
-	var domainInvalidMsg = ["\nPlease use a domain name instead of an IP address.", "\nDomain is not valid. Please use a valid domain name."];
-	var portInvalidMsg = ["\nPort should be a number.", "\nPort should be a number from 1 and 65535."];
+	const domainInvalidMsg = ["\nPlease use a domain name instead of an IP address.", "\nDomain is not valid. Please use a valid domain name."];
+	const portInvalidMsg = ["\nPort should be a number.", "\nPort should be a number from 1 and 65535."];
 	
 	// ARGV returns a string as input. Port numbers should be parsed to int to validate them. If validation fails, these will return undefined and will fail the subsequent test.
-	var validInPort = parseToInt(inPort);
-	var validOutPort = parseToInt(outPort);
+	const validInPort = parseToInt(inPort);
+	const validOutPort = parseToInt(outPort);
 
 	// The value of isInvalid will be returned back. If none of the `if`s are true, the default value `true` is returned `domain`, `inPort` and `outPort` are considered validated.
-	var isValid = true;
+	let isValid = true;
 
 	// Throw if IP is given instead of domain name.
 	if (isIP(domain)) {
@@ -60,4 +62,4 @@ function validate(domain, inPort, outPort) {
 	}
 }
 
-module.exports = validate;
+module.exports = validate;