Rollbar ignores 4xx errors

Fixes rb#99, #135

application/index.jsx

@@ -78,7 +78,9 @@ app.get(/^(?!\/(?:(?:api|svg)\/|png|zip))[^.]*$/, function(req, res, next) {
 				return res.redirect(302, response[0].pathname + response[0].search);
 			}
 			if (!response[1]) {
-				return res.status(404).send('Not Found');
+				var err = new Error();
+				err.status = 404;
+				return next(err);
 			}
 
 			var store = Store();
@@ -94,7 +96,9 @@ app.get(/^(?!\/(?:(?:api|svg)\/|png|zip))[^.]*$/, function(req, res, next) {
 						console.timeEnd('time_' + i);
 					}
 					if (!html) {
-						return res.status(404).send('Not Found');
+						var err = new Error();
+						err.status = 404;
+						return next(err);
 					}
 					res.set('Cache-Control', 'public, max-age=' + (24 * 60 * 60));
 					res.set('Last-Modified', (new Date()).toUTCString());

index.js

@@ -2,6 +2,7 @@ require('babel-register');
 var _         = require('underscore');
 var feathers  = require('feathers');
 var fs        = require('fs');
+var http      = require('http');
 var logos     = require('instant-logos');
 var os        = require('os');
 var path      = require('path');
@@ -25,6 +26,14 @@ var search      = require('./search');
 var Logos       = require('./services/Logos');
 var Suggestions = require('./services/Suggestions');
 
+var rollbarErrorHandler = (process.env.NODE_ENV && process.env.ROLLBAR_ACCESS_TOKEN) ?
+	rollbar.errorHandler(process.env.ROLLBAR_ACCESS_TOKEN || ' ', rollbar.config) :
+	function(err, req, res, next) {
+		console.error(err);
+		console.error(err.stack);
+		next(err);
+	};
+
 _.chain(logos)
 	.reject(function(logo) {
 		return !logo.svg || !logo.svg.path;
@@ -161,23 +170,26 @@ app.get('/api/logo_suggestions', function(req, res, next) {
 	);
 });
 
-app.all('*', function(req, res) {
-	res.status(404).send('Not Found');
+app.all('*', function(req, res, next) {
+	var err = new Error();
+	err.status = 404;
+	next(err);
 });
 
-app.use((process.env.NODE_ENV && process.env.ROLLBAR_ACCESS_TOKEN) ?
-	rollbar.errorHandler(process.env.ROLLBAR_ACCESS_TOKEN || ' ', rollbar.config) :
-	function(err, req, res, next) {
-		console.error(err);
-		console.error(err.stack);
-		next(err);
-	});
+app.use(function(err, req, res, next) {
+	var status = err.status || err.code || 500;
+	if (status >= 400 && status < 500) {
+		return next(err);
+	}
+	rollbarErrorHandler(err, req, res, next);
+});
 
 app.use(function(err, req, res, next) {
 	if (res.headersSent) {
 		return next(err);
 	}
-	res.status(500).send(err.message);
+	var status = err.status || err.code || 500;
+	res.status(status).send(err.message || http.STATUS_CODES[status]);
 });
 
 app.listen(app.get('port'), function() {