Options should all be implemented.

Author: jfhbrook

lib/http-server.js

@@ -10,8 +10,8 @@ var opts = {
   root: argv._[0] || "./public",
   port: argv.p || 8080,
   host: argv.a || 'localhost',
-  cache: argv.c || 3600, // in seconds. TODO: add configurability to ecstatic
-  autoIndex: argv.i || true, // TODO: add feature/configurability to ecstatic
+  cache: argv.c || 3600, // in seconds.
+  autoIndex: argv.i || true,
   silent: argv.s || argv.silent || false,
   help: argv.h || argv.help
 }
@@ -39,7 +39,12 @@ server.log = function (message) {
 }
 
 
-server.start = function () {
+server.start = function (overrides) {
+  if (overrides) {
+    Object.keys(overrides).forEach(function (k) {
+      opts[k] = overrides[k];
+    });
+  }
 
   if (opts.help) {
     return showHelp();
@@ -48,8 +53,10 @@ server.start = function () {
   //TODO: Add 404 file behavior to ecstatic, make configurable
   app.use(flatiron.plugins.http, {
     before: [
-      ecstatic(opts.root),
-      ecstatic.showDir(opts.root)
+      ecstatic(opts.root, {
+        autoIndex: opts.autoIndex,
+        cache: opts.cache
+      })
     ]
   });
 

package.json

@@ -23,7 +23,10 @@
   "dependencies" : {
     "eyes": "~0.1.6",
     "colors" : "*",
-    "optimist" : "0.2.x"
+    "optimist" : "0.2.x",
+    "flatiron": "0.1.x",
+    "union": "0.1.x",
+    "ecstatic": "0.1.x"
   },
   "analyze": false,
   "devDependencies": {

test/http-server-test.js

@@ -3,18 +3,17 @@ var assert = require('assert'),
     fs = require('fs'),
     vows = require('vows'),
     request = require('request'),
-    HTTPServer = require('../lib/http-server').HTTPServer;
+    httpServer = require('../lib/http-server');
 
 var root = path.join(__dirname, 'fixtures', 'root');
 
 vows.describe('http-server').addBatch({
   'When http-server is listening on 8080': {
     topic: function () {
-      var httpServer = new HTTPServer({
+      httpServer.start({
         port: 8080,
         root: root
       });
-      httpServer.start();
       this.callback(null, httpServer);
     },
     'it should serve files from root directory': {