fixed restarts caused by 404s of unexpected http verbs

HISTORY.md

@@ -2,7 +2,8 @@
 
 ## 0.6.9 
 
-- Fixed bug where `internal-client` was not accessible from modules
+ - Fixed bug where `internal-client` was not accessible from modules
+ - Fixed restarts caused by 404s of unexpected http verbs
 
 ## 0.6.8
 

lib/router.js

@@ -3,6 +3,7 @@ var db = require('./db')
   , escapeRegExp  = /[\-\[\]{}()+?.,\\\^$|#\s]/g
   , debug = require('debug')('router')
   , doh = require('doh')
+  , error404 = doh.createResponder()
   , async = require('async');
 
 /**
@@ -83,7 +84,7 @@ Router.prototype.route = function (req, res) {
         } else {
           debug('404 %s', req.url);
           res.statusCode = 404;
-          res.domain.emit('error', new Error('Resource not found'));
+          error404({message: 'resource not found'}, req, res);
         }
       });
     });

test-app/public/index.html

@@ -12,6 +12,7 @@
   <script>mocha.setup('bdd')</script>
   <!-- -- tests -- -->
   <script src="test/util.test.js"></script>
+  <script src="test/misc.test.js"></script>
   <script src="test/collection.test.js"></script>
   <script src="test/user-collection.test.js"></script>
 </head>

test-app/public/test/misc.test.js

@@ -0,0 +1,17 @@
+describe('404s', function(){
+  it('should not prevent a server from responding', function(done) {
+    this.timeout(5000);
+
+    var tests = 50;
+    var remaining = tests;
+
+    while(tests--) {
+      dpd('/').post({foo: 'bar'}, function (res, err) {
+        remaining--;
+        if(!remaining) {
+          done();
+        }
+      });
+    }
+  });
+});