From fad45fb654378de5d02ed99e0956eba898ad20c7 Mon Sep 17 00:00:00 2001
From: Michael Jackson <mjijackson@gmail.com>
Date: Sat, 22 Dec 2012 11:29:54 -0800
Subject: [PATCH] Small style tweaks

---
 lib/common-logger.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/lib/common-logger.js b/lib/common-logger.js
index 8a45440..bf9fea0 100644
--- a/lib/common-logger.js
+++ b/lib/common-logger.js
@@ -17,22 +17,23 @@ module.exports = function (app, stream) {
       var req = new Request(env);
 
       var addr = req.remoteAddr || '-';
-      var id = '-';
+      var id = '-'; // RFC 1413 identity of the client determined by identd on the client's machine
       var user = env.remoteUser || '-';
       var timestamp = '[' + strftime('%d/%b/%Y:%H:%M:%S %z', env.requestTime) + ']';
-      var protoVersion = 'HTTP/' + env.protocolVersion;
-      var reqline = '"' + req.method + ' ' + req.fullPath + ' ' + protoVersion  + '"';
-      var length = '-';
+      var info = '"' + req.method + ' ' + req.fullPath + ' HTTP/' + env.protocolVersion  + '"';
 
+      var length;
       if ('Content-Length' in headers) {
         length = headers['Content-Length'];
       } else if (typeof body === 'string') {
         length = Buffer.byteLength(body);
       } else if (typeof body.length === 'number') {
-        length = String(body.length);
+        length = body.length;
+      } else {
+        length = '-';
       }
 
-      var entry = [addr, id, user, timestamp, reqline, status, length].join(' ');
+      var entry = [addr, id, user, timestamp, info, status, length].join(' ');
       stream.write(entry + '\n');
 
       callback(status, headers, body);