diff --git a/.eslintrc.js b/.eslintrc.js index 095799d..8b6802a 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -15,10 +15,7 @@ module.exports = { "prefer-const": 1, // 4 errors "no-unused-vars": 1, // 3 errors "no-use-before-define": 1, // 7 errors - "one-var": 1, // 8 errors "consistent-return": 1, // 8 errors - "no-param-reassign": 1, // 2 errors - "global-require": 1, // 2 errors "import/no-dynamic-require": 1, // 1 error "comma-dangle": ["error", "never"] } diff --git a/lib/cli/create.js b/lib/cli/create.js index 7ba3c46..234065f 100644 --- a/lib/cli/create.js +++ b/lib/cli/create.js @@ -17,21 +17,21 @@ const start = require('./start'); * Create a new app with folder structure and files */ const create = function (name) { - name = name || 'my-deployd-app'; - if (test('-d', name)) { - return console.info(`${name} already exists in this directory`); + const newProjectName = name || 'my-deployd-app'; + if (test('-d', newProjectName)) { + return console.info(`${newProjectName} already exists in this directory`); } - mkdir('-p', name); - cp('-Rf', path.join(__dirname, '../createtemplate/*'), name); - mkdir('-p', `${name}/.dpd`); - mkdir('-p', `${name}/.dpd/pids`); - ('').to(`${name}/.dpd/pids/mongod`); + mkdir('-p', newProjectName); + cp('-Rf', path.join(__dirname, '../createtemplate/*'), newProjectName); + mkdir('-p', `${newProjectName}/.dpd`); + mkdir('-p', `${newProjectName}/.dpd/pids`); + ('').to(`${newProjectName}/.dpd/pids/mongod`); ('# npm\r\nnode_modules/\r\nnpm-debug.log\r\n\r\n# deployd\r\n.dpd/\r\ndata/') - .to(`${name}/.gitignore`); - rm(ls('-R', name).filter(p => path.basename(p) === 'PLACEHOLDER').map(p => `${name}/${p}`)); + .to(`${newProjectName}/.gitignore`); + rm(ls('-R', newProjectName).filter(p => path.basename(p) === 'PLACEHOLDER').map(p => `${newProjectName}/${p}`)); - process.chdir(name); + process.chdir(newProjectName); console.log('dpd is installing the dependencies... please be patient (this may take a few minutes)'); const child = exec('npm install', @@ -40,10 +40,10 @@ const create = function (name) { console.log(stderr); console.log(`npm install error: ${error}`); } else if (program.dashboard || program.open) { - start(`${name}/app.dpd`); + start(`${newProjectName}/app.dpd`); } else { console.info('to start your app:'); - console.info('\t$ cd', name); + console.info('\t$ cd', newProjectName); console.info('\t$ dpd'); } }); diff --git a/lib/cli/keygen.js b/lib/cli/keygen.js index f7f39e9..2707256 100644 --- a/lib/cli/keygen.js +++ b/lib/cli/keygen.js @@ -1,11 +1,12 @@ 'use strict'; +const Keys = require('../keys'); + /** * Generate a key */ const keygen = function () { - const Keys = require('../keys'), - keys = new Keys(); + const keys = new Keys(); keys.create((err, key) => { if (err) return console.error(err); diff --git a/lib/cli/showkey.js b/lib/cli/showkey.js index aaf262a..0c320e4 100644 --- a/lib/cli/showkey.js +++ b/lib/cli/showkey.js @@ -1,11 +1,12 @@ 'use strict'; +const Keys = require('../keys'); + /** * Show current key */ const showkey = function () { - const Keys = require('../keys'), - keys = new Keys(); + const keys = new Keys(); keys.getLocal((err, key) => { if (err) return console.error(err); diff --git a/lib/cli/start.js b/lib/cli/start.js index 98a4369..61c33c6 100644 --- a/lib/cli/start.js +++ b/lib/cli/start.js @@ -28,13 +28,13 @@ const createServer = require('./createserver'); * Start the server */ const start = function (file) { - let port = program.port, - host = program.host || '127.0.0.1', - dbname = program.dbname || '-deployd', - mongoPort = generatePort(), - env = program.environment || process.env.DPD_ENV || 'development', - retries = 0, - credentials = {}; + let port = program.port; + const host = program.host || '127.0.0.1'; + const dbname = program.dbname || '-deployd'; + const mongoPort = program.mongoPort ? Number(program.mongoPort) : '27017'; + const env = program.environment || process.env.DPD_ENV || 'development'; + let retries = 0; + const credentials = {}; if (!port) { @@ -42,10 +42,6 @@ const start = function (file) { retries = env === 'development' && 5; } - if (program.mongoPort) { - mongoPort = Number(program.mongoPort); - } - if (file) { process.chdir(path.dirname(file)); } @@ -98,9 +94,9 @@ const start = function (file) { if (program.host) { if (program.auth) { - const auth = program.auth.split(':'), - username = auth[0], - password = auth[1]; + const auth = program.auth.split(':'); + const username = auth[0]; + const password = auth[1]; setCredentials(username, password); } else if (program.username || program.password) { setCredentials(program.username, program.password); @@ -170,14 +166,6 @@ const start = function (file) { } }; -/** - * Port generation - */ -function generatePort() { - const portRange = [3000, 9000]; - return Math.floor(Math.random() * (portRange[1] - portRange[0])) + portRange[0]; -} - function checkForUpdates() { http.get('http://registry.npmjs.org/deployd-cli', (err, res, body) => { if (!err) { diff --git a/lib/keys.js b/lib/keys.js index 49c2425..4d9c1db 100644 --- a/lib/keys.js +++ b/lib/keys.js @@ -35,14 +35,15 @@ Keys.prototype.generate = function () { */ Keys.prototype.create = function (fn) { - const key = this.generate(), - keys = this; + const key = this.generate(); + const keys = this; this.readFile((err, data) => { if (err) return fn(err); - data[key] = true; - keys.writeFile(data, (errW) => { + const fileData = data; + fileData[key] = true; + keys.writeFile(fileData, (errW) => { fn(errW, key); }); }); @@ -54,8 +55,8 @@ Keys.prototype.create = function (fn) { Keys.prototype.readFile = function (fn) { fs.readFile(this.path, 'utf-8', (err, data) => { - let jsonData, - error; + let jsonData; + let error; try { jsonData = (data && JSON.parse(data)) || {};