Implemented artifacts list; commit status URLs supported

BuildServer/app.js

@@ -40,8 +40,9 @@ app.get('/', routes.index);
 app.post('/github/postreceive', routes.postreceive);
 app.get('/manual', routes.manual.get);
 app.post('/manual', routes.manual.post);
-app.get('/status/:owner/:reponame/:branch/:rev', routes.status.page);
+app.get('/status/:owner/:reponame/:branch/:rev?', routes.status.page);
 app.get('/status.svg', routes.status.image);
+app.get('/artifact/:owner/:reponame/:branch/:rev/:file', routes.artifact);
 
 http.createServer(app).listen(app.get('port'), function(){
   console.log('Express server listening on port ' + app.get('port'));

BuildServer/routes/status.js

@@ -3,53 +3,68 @@
 var fs = require('fs'),
 	url = require('url');
 
-var parseOptionsFromReferer = function (req, callback) {
-	var pathParts = (url.parse(req.headers.referer || "").pathname || "").split("/");
-	var result = {};
-	if (pathParts.length < 3) {
-		return callback("BadRequest", result);
-	}
-
-	result.owner = pathParts[1];
-	result.reponame = pathParts[2];
-	if (pathParts[3] && /^[\da-f]{40}$/i.test(pathParts[4])) {
-		result.rev = pathParts[4];
-		var branchFile = req.app.get('releasepath') + "/" + result.owner + "/" + result.reponame + "/$revs/" + result.rev + ".branch";
+var addBranchInfo = function (app, options, callback) {
+	var branchFile = app.get('releasepath') + "/" + options.owner + "/" + options.reponame + "/$revs/" + options.rev + ".branch";
 	fs.exists(branchFile, function (exists) {
 		if (!exists) {
-				return callback("BranchFileNotFound", result);
+			return callback("BranchFileNotFound", options);
 		}
 		fs.readFile(branchFile, function (err, data) {
 			if (err) {
 				return callback(err, result);
 			}
-				result.branch = data.toString();
-				result.branchName = result.branch.split("/").pop();
-				return callback(null, result);
+			options.branch = data.toString();
+			options.branchName = options.branch.split("/").pop();
+			return callback(null, options);
 		});
 	});
-	} else {
-		result.branchName = pathParts[4] || "master";
-		result.branch = "refs/heads/" + result.branchName;
-//		console.log(result);
-		var revFile = req.app.get('releasepath') + "/" + result.owner + "/" + result.reponame + "/" + result.branch + "/latest.id";
+};
+
+var addRevInfo = function (app, options, callback) {
+	var revFile = app.get('releasepath') + "/" + options.owner + "/" + options.reponame + "/" + options.branch + "/latest.id";
 	fs.exists(revFile, function (exists) {
 		if (!exists) {
-				return callback("RevFileNotFound", result);
+			return callback("RevFileNotFound", options);
 		}
 		fs.readFile(revFile, function (err, data) {
 			if (err) {
-					return callback(err, result);
+				return callback(err, options);
 			}
-				result.rev = data.toString();
-				return callback(null, result);
+			options.rev = data.toString();
+			return callback(null, options);
 		});
 	});
+};
+
+var parseOptionsFromReferer = function (req, callback) {
+	var pathParts = (url.parse(req.headers.referer || "").pathname || "").split("/");
+	var result = {};
+	if (pathParts.length < 3) {
+		return callback("BadRequest", result);
+	}
+
+	result.owner = pathParts[1];
+	result.reponame = pathParts[2];
+	if (pathParts[3] && /^[\da-f]{40}$/i.test(pathParts[4])) {
+		result.rev = pathParts[4];
+		return addBranchInfo(req.app, result, callback);
+	} else {
+		result.branchName = pathParts[4] || "master";
+		result.branch = "refs/heads/" + result.branchName;
+		return addRevInfo(req.app, result, callback);
 	}
 };
 
 var loadReport = function (app, options, callback) {
-	var reportFile = app.get('releasepath') + "/" + options.owner + "/" + options.reponame + "/" + options.branch + "/" + options.rev + "/report.json";
+	var releaseDir = app.get('releasepath') + "/" + options.owner + "/" + options.reponame + "/" + options.branch + "/" + options.rev;
+
+	fs.readdir(releaseDir, function (err, files) {
+		if (err) {
+			return callback(err, options);
+		}
+
+		var reportFile = reportFile = releaseDir + "/report.json";
+		options.files = files;
 		fs.exists(reportFile, function (exists) {
 			if (!exists) {
 				return callback("ReportFileNotFound", options);
@@ -67,10 +82,11 @@ var loadReport = function (app, options, callback) {
 				return callback(null, options);
 			});
 		});
+	});
 };
 
 exports.image = function(req, res) {
-//	console.log(req.headers);
+	console.log(req.headers);
 	var handle = function (err, options) {
 		if (err === "ReportFileNotFound") {
 			options.status = "Building";
@@ -116,9 +132,20 @@ exports.page = function(req, res) {
 		branch: "/refs/heads/" + req.params.branch,
 		rev: req.params.rev
 	};
+	var finish = function (err, options) {
 		loadReport(req.app, options, function (err, options) {
 			options.err = err;
 			console.log(options);
 			res.render('status', options);
 		});
 	};
+
+	if (/^[\da-f]{40}$/i.test(options.branchName)) {
+		options.rev = options.branchName;
+		options.branchName = undefined;
+		options.branch = undefined;
+		return addBranchInfo(req.app, options, finish);
+	} else {
+		return finish(undefined, options);
+	}
+};