fixed issue with no host header set

longshoreman · Oct 2, 2014 · fa43b5f · fa43b5f
1 parent 02d8139
commit fa43b5f
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -1,4 +1,4 @@
-version = $(npm version)
+version = $(shell cat package.json | jq -r '.version')
 image = longshoreman/longshoreman:$(version)
 
 all: build push

diff --git a/index.js b/index.js
@@ -14,7 +14,11 @@ var PORT = process.env.PORT || 3000;
 var app = express();
 
 app.use(function(req, res, next) {
-  var hostname = req.get('host').split(':')[0];
+  var host = req.get('host');
+  if (!host) {
+    return res.status(400).send('No host header');
+  }
+  var hostname = host.split(':')[0];
   if (process.env.CONTROLLER_HOST == hostname) {
     controller(req, res, next);
   } else {

diff --git a/package.json b/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "docker-deploy",
+  "name": "longshoreman",
   "version": "0.9.1",
   "description": "Deploy Docker containers in the clouds",
   "main": "index.js",