fix: properly handle non-errors thrown in domains

Author: DonutEspresso

lib/server.js

@@ -1309,13 +1309,19 @@ Server.prototype._routeErrorResponse = function _routeErrorResponse(
             return;
         }
 
-        // Expose only knonw errors
-        if (_.isNumber(err.statusCode)) {
+        // only automatically send errors that are known (e.g., restify-errors)
+        if (err && _.isNumber(err.statusCode)) {
             res.send(err);
             return;
         }
 
-        res.send(new errors.InternalError(emitErr || err));
+        var finalErr = emitErr || err;
+        if (!(finalErr instanceof Error)) {
+            // if user land threw domain or string or numbers, handle it here
+            // as best we can.
+            finalErr = (finalErr && finalErr.toString()) || '';
+        }
+        res.send(new errors.InternalError(finalErr));
     });
 };
 

test/server.test.js

@@ -2505,9 +2505,10 @@ test('uncaughtException should handle thrown undefined literal', function(t) {
         }
     );
 
-    CLIENT.get('/foo', function(err, _, res) {
+    CLIENT.get('/foo', function(err, _, res, data) {
         t.ok(err);
         t.equal(res.statusCode, 500);
+        t.equal(data.message, '');
         t.end();
     });
 });
@@ -2535,8 +2536,9 @@ test('uncaughtException should handle thrown Number', function(t) {
         }
     );
 
-    CLIENT.get('/foo', function(err, _, res) {
+    CLIENT.get('/foo', function(err, _, res, data) {
         t.ok(err);
+        t.equal(data.message, '1');
         t.equal(res.statusCode, 500);
         t.end();
     });