Merge pull request #11 from connyay/enable-http-checking

enable basic http & https checking

Author: lphiri

bin/dockerfile_lint

@@ -39,6 +39,29 @@ function printEntry(entry, level){
 
 }
 
+function isRedirect(statusCode) {
+  return (statusCode === 300 || statusCode === 301 || statusCode === 302);
+}
+
+function downloadDockerfile(url, cb) {
+  var proto, dockerfile = '';
+  if(url.match('^https://')) {
+    proto = require('https');
+  } else {
+    proto = require('http');
+  }
+  proto.get(url, function(res) {
+    res.on('data', function(data) {
+        dockerfile += data;
+    }).on('end', function() {
+        if (isRedirect(res.statusCode) && res.headers.location) {
+          return downloadDockerfile(res.headers.location, cb);
+        }
+        cb(dockerfile);
+    });
+  });
+}
+
 function printResults(results){
   var errors = results.error;
   var warn = results.warn;
@@ -80,6 +103,7 @@ var rulefileLocation = null;
 var dockerfile = null;
 var rulefile = null;
 var printJson = false;
+var remoteFile = false;
 
 
 
@@ -104,8 +128,12 @@ if (commandline.rulefile){
 try {
   dockerfile = fs.readFileSync(dockerfileLocation, 'UTF-8');
 } catch (e) {
-  console.error('ERROR: Dockerfile not found -> ' + dockerfileLocation);
-  process.exit(1);
+  if (/^http[s]?:\/\//.test(dockerfileLocation)) {
+    remoteFile = true;
+  } else {
+    console.error('ERROR: Dockerfile not found -> ' + dockerfileLocation);
+    process.exit(1);
+  }
 }
 if (rulefileLocation !== null) {
   try {
@@ -116,16 +144,27 @@ if (rulefileLocation !== null) {
   }
 }
 
-var validator = new DockeFileValidator(rulefile);
-var results = validator.validate(dockerfile);
-if (printJson){
-   printJsonResults(results);
-}else {
-   printResults(results);
+function runValidation(dockerfile, rulefile) {
+  var validator = new DockeFileValidator(rulefile);
+  var results = validator.validate(dockerfile);
+  if (printJson){
+     printJsonResults(results);
+  }else {
+     printResults(results);
+  }
+
+  if (results.error.count > 0) {
+      process.exit(1);
+  } else {
+      process.exit(0);
+  }
 }
 
-if (results.error.count > 0) {
-    process.exit(1);
+if (remoteFile) {
+  downloadDockerfile(dockerfileLocation, function(dockerfile){
+    runValidation(dockerfile, rulefile);
+  });
 } else {
-    process.exit(0);
+  runValidation(dockerfile, rulefile);
 }
+