errors should just set a reason header

esatterwhite · Dec 29, 2016 · 2f38bc7 · 2f38bc7
1 parent 1493c70
commit 2f38bc7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/server/response.js b/lib/server/response.js
@@ -11,17 +11,15 @@ Response.prototype.error = function error( err, msg ) {
       message: msg 
     })
   }
-
   err.statusCode = err.statusCode || err.code;
   if( !err.statusCode ) {
     err.statusCode = 500;
     err.message = 'Internal Server Error';
   }
 
   this.status( err.statusCode );
-  return this.json({
-    message: err.message
-  });
+  this.res.setHeader('x-reason', err.message)
+  return this.end()
 }
 
 Response.prototype.get = function get( key ) {

diff --git a/lib/timer.js b/lib/timer.js
@@ -11,7 +11,7 @@
  * @requires skyring/lib/json
  */
 const transports = require('./transports')
-    , nats      = require('./nats')
+    , nats       = require('./nats')
     , json       = require('./json')
     , debug      = require('debug')('skyring:timer')
     , rebalance  = require('debug')('skyring:rebalance')