[fix] Properly observe the -s|--silent option for request logging. …

…Bug in #44.
http-party · Dec 6, 2013 · 2b0b559 · 2b0b559
1 parent a8d4a8b
commit 2b0b559
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/bin/http-server b/bin/http-server
@@ -29,12 +29,11 @@ var port = argv.p,
     log = (argv.s || argv.silent) ? (function () {}) : console.log,
     requestLogger;
 
- if (log) {
-   requestLogger = function(req) {
-      var d = new Date();
-      log('[%s] "%s %s" "%s"', (new Date).toUTCString(), req.method.cyan, req.url.cyan, req.headers['user-agent']);
-   }
- }
+if (!argv.s && !argv.silent) {
+  requestLogger = function(req) {
+    log('[%s] "%s %s" "%s"', (new Date).toUTCString(), req.method.cyan, req.url.cyan, req.headers['user-agent']);
+  }
+}
 
 if (!argv.p) {
   portfinder.basePort = 8080;