From 8a631e70f732f7d4fc95d39fe719e93d39a59b02 Mon Sep 17 00:00:00 2001
From: Shane Kilkelly <shane@kilkelly.me>
Date: Fri, 28 Aug 2015 12:16:41 +0100
Subject: [PATCH] Refactor, and log when we have a 'range' header.

---
 services/filestore/app/coffee/FileController.coffee | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/services/filestore/app/coffee/FileController.coffee b/services/filestore/app/coffee/FileController.coffee
index 22db596d94..9cb01b73ea 100644
--- a/services/filestore/app/coffee/FileController.coffee
+++ b/services/filestore/app/coffee/FileController.coffee
@@ -13,19 +13,20 @@ module.exports = FileController =
 	getFile: (req, res)->
 		{key, bucket} = req
 		{format, style} = req.query
-		opts = {
+		options = {
 			key: key,
 			bucket: bucket,
 			format: format,
 			style: style,
 		}
 		metrics.inc "getFile"
+		logger.log key:key, bucket:bucket, format:format, style: style, "reciving request to get file"
 		if req.headers.range?
 			range = FileController._get_range(req.headers.range)
-			opts.start = range.start
-			opts.end = range.end
-		logger.log key:key, bucket:bucket, format:format, style:style, "reciving request to get file"
-		FileHandler.getFile bucket, key, opts, (err, fileStream)->
+			options.start = range.start
+			options.end = range.end
+			logger.log start: range.start, end: range.end, "getting range of bytes from file"
+		FileHandler.getFile bucket, key, options, (err, fileStream)->
 			if err?
 				logger.err err:err, key:key, bucket:bucket, format:format, style:style, "problem getting file"
 				if !res.finished and res?.send?