Re-introduce EXPOSE directives to Dockerfile.

Author: RickCarlino

Dockerfile

@@ -1,7 +1,12 @@
 FROM ubuntu
 MAINTAINER [email protected]
 
-# ENV WEB_API_URL https://enigmatic-meadow-89512.herokuapp.com 
+EXPOSE 1883
+EXPOSE 8883
+EXPOSE 80
+EXPOSE 443
+EXPOSE 3002
+
 RUN apt-get update
 RUN apt-get install -y curl
 

app/config.js

@@ -1,21 +1,19 @@
+var log = require("./logger");
+
 // TODO: Clean up ENV management into something less crappy.
 // or use a 3rd party node ENV manager module.
+var DEFAULT_URL = "http://localhost:3000";
 
 var webAppUrl;
-var log = require("./logger");
-
 if (process.env.WEB_API_URL) {
-  webAppUrl = process.env.WEB_API_URL;
+    webAppUrl = process.env.WEB_API_URL;
 } else {
-  missing("WEB_API_URL");
+    webAppUrl = DEFAULT_URL;
+    log("You did not set WEB_API_URL. Defaulting to " + DEFAULT_URL);
 }
 
 module.exports = {
-  webAppUrl: webAppUrl,
-  httpPort: 3002,
-  mqttPort: 1883
-}
-
-function missing(varName) {
-  log("You did not set the `" + varName + "` ENV var.");
+    webAppUrl: webAppUrl,
+    httpPort: 3002,
+    mqttPort: 1883
 }

app/index.js

@@ -11,6 +11,7 @@ var input = {
         static: "./public"
     }
 };
+
 maybeEnableSSL(input)
 var server = new Server(input);
 

package.json

@@ -5,7 +5,7 @@
   "main": "app/index.js",
   "scripts": {
     "start": "node app/index.js",
-    "test": "WEB_API_URL=http://localhost:3000 node_modules/.bin/istanbul cover node_modules/.bin/jasmine",
+    "test": "node_modules/.bin/istanbul cover node_modules/.bin/jasmine",
     "dev": "WEB_API_URL=http://localhost:3000 node app/index.js"
   },
   "repository": {
@@ -28,4 +28,4 @@
     "istanbul": "^0.4.4",
     "jasmine": "^2.4.1"
   }
-}
+}