Append port number to config files and webroot

actions/createProxyServer.js

@@ -8,11 +8,10 @@ var nginxReload = require('../util/nginxReload');
 var { EOL } = require('os'); // \n if used on Linux, \r\n if used on Windows.
 
 function createProxyServer(domain, inPort, outPort) {
-    fs.outputFileSync((conf(npath.availableSites(), domain)),
+	fs.outputFileSync((conf(npath.availableSites(), domain, outPort)),
 		"server {" + EOL +
 		"	listen " + outPort + ";" + EOL +
 		"	listen [::]:" + outPort + ";" + EOL +
-        "	root /var/www/" + domain + ";" + EOL +
 		"	index index.html index.htm;" + EOL +
 		"" + EOL +
 		"	server_name " + domain + ";" + EOL +
@@ -27,7 +26,7 @@ function createProxyServer(domain, inPort, outPort) {
 		"}"
 	)
 	shell.mkdir('-p', npath.enabledSites()); // Creates directory if doesn't exist
-    shell.ln('-sf', conf(npath.availableSites(), domain), conf(npath.enabledSites(), domain)); // Symlink the conf file from sites-available to sites-enabled
+	shell.ln('-sf', conf(npath.availableSites(), domain, outPort), conf(npath.enabledSites(), domain, outPort)); // Symlink the conf file from sites-available to sites-enabled
 
 	nginxReload();
 };

actions/createStaticServer.js

@@ -10,7 +10,7 @@ var currentPath = path.normalize(process.cwd());
 var { EOL } = require('os'); // \n if used on Linux, \r\n if used on Windows.
 
 function createStaticServer(domain, outPort = 80) {
-	fs.outputFileSync((conf(npath.availableSites(), domain)), // Gets nginx's paths from nginxPath.js
+	fs.outputFileSync((conf(npath.availableSites(), domain, outPort)), // Gets nginx's paths from nginxPath.js
 		"server {" + EOL +
 		"	listen " + outPort + ";" + EOL +
 		"	listen [::]:" + outPort + ";" + EOL +
@@ -24,11 +24,11 @@ function createStaticServer(domain, outPort = 80) {
 		"}"
 		)
 		shell.mkdir('-p', npath.enabledSites()); // Creates directory if doesn't exist
-		shell.rm('-rf', conf(npath.enabledSites(), domain)); // Removes domain from sites-enabled if exists
+		shell.rm('-rf', conf(npath.enabledSites(), domain, outPort)); // Removes domain from sites-enabled if exists
-		shell.ln('-sf', conf(npath.availableSites(), domain), conf(npath.enabledSites(), domain)); // Symlink the conf file from sites-available to sites-enabled
+		shell.ln('-sf', conf(npath.availableSites(), domain, outPort), conf(npath.enabledSites(), domain, outPort)); // Symlink the conf file from sites-available to sites-enabled
-		shell.rm('-rf', npath.webRoot() + domain); // Removes domain from webroot if exists
+		shell.rm('-rf', npath.webRootDomain(domain, outPort)); // Removes domain from webroot if exists
 		shell.mkdir('-p', npath.webRoot()); // Creating the nginx www path if it doesn't exist so symlink doesn't fail
-		shell.ln('-sf', currentPath, npath.webRoot() + domain); // Symlink current directory to nginx's web root
+		shell.ln('-sf', currentPath, npath.webRootDomain(domain, outPort)); // Symlink current directory to nginx's web root
 
 		nginxReload();
 };

actions/killServer.js

@@ -4,10 +4,10 @@ var npath = require('../util/nginxPath');
 var conf = require('../util/nginxConf');
 var nginxReload = require('../util/nginxReload');
 
-function killServer(domain) {
+function killServer(domain, outPort) {
-	shell.rm('-rf', conf(npath.enabledSites(), domain));
+	shell.rm('-rf', conf(npath.enabledSites(), domain, outPort));
-	shell.rm('-rf', conf(npath.availableSites(), domain));
+	shell.rm('-rf', conf(npath.availableSites(), domain, outPort));
-	shell.rm('-rf', npath.webRoot() + domain);
+	shell.rm('-rf', npath.webRootDomain(domain, outPort));
 
 	nginxReload();
 }

dev.com.undefined

@@ -0,0 +1 @@
+Subproject commit 965be8551443f1a797ec528eb37458547f895d1b

index.js

@@ -16,10 +16,10 @@ var createStaticServer = require('./actions/createStaticServer');
 var killServer = require('./actions/killServer');
 
 // 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.
-requirements(); // Comment in development and uncomment this line in production. This should check whether the OS is compatible with this version of `up`
+// requirements(); // Comment in development and uncomment this line in production. This should check whether the OS is compatible with this version of `up`
 
 program
-	.version('0.1.2');
+	.version('0.1.5');
 
 program
 	.command('static <domain> [outPort]')
@@ -47,10 +47,10 @@ program
 	});
 
 program
-	.command('kill <domain>')
+	.command('kill <domain> [ourPort]')
 	.description('Kill a server.')
-	.action(function (domain) {
+	.action(function (domain, outPort = 80) {
-		killServer(domain);
+		killServer(domain, outPort);
 		console.log("Done! Your server has been killed!\n");
 	});
 

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "up-serve",
-	"version": "0.1.4",
+	"version": "0.1.5",
 	"description": "A cli tool to quickly create and manage nginx server blocks.",
 	"main": "index.js",
 	"scripts": {

util/nginxConf.js

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

util/nginxPath.js

@@ -1,20 +1,27 @@
 // These functions just return paths. Later, these should be modified to poll from nginx's config.
 
+var available = "/etc/nginx/sites-available/";
+var enabled = "/etc/nginx/sites-enabled/";
+var wwwRoot = "/var/www/";
+
 function availableSites() {
-	var availableSites = "/etc/nginx/sites-available/";
+	return available;
-	return availableSites;
 }
 
 function enabledSites() {
-	var enabledSites = "/etc/nginx/sites-enabled/";
+	return enabled;
-	return enabledSites;
 }
 
 function webRoot() {
-	var webRoot = "/var/www/";
+	return wwwRoot;
-	return webRoot;
+}
+
+function webRootDomain(domain, outPort) {
+	rootWithDomain = wwwRoot + domain + "." + outPort;
+	return rootWithDomain;
 }
 
 module.exports.availableSites = availableSites;
 module.exports.enabledSites = enabledSites;
 module.exports.webRoot = webRoot;
+module.exports.webRootDomain = webRootDomain;