Merge pull request #97 from SparkartGroupInc/sentry-integration

Sentry integration

Author: takempf

lib/preprocessor.js

@@ -21,6 +21,7 @@ var Preprocessor = function( preprocessor_path, options ){
 
 	var server = options.server;
 	var logger = server.logger;
+	var raven_client = server.raven_client;
 	var preprocessor = this;
 	var data = this.data = '{}';
 	this.path = preprocessor_path;
@@ -29,8 +30,17 @@ var Preprocessor = function( preprocessor_path, options ){
 	this.process = function( context, callback ){
 
 		workers( context, this.path, function( err, preprocessed_context ){
-			if( err ){ 
+			// preprocessor errors don't bubble up
+			// so log them here
+			if( err ){
 				logger.log( 'Preprocessor Error:\n'+ err.stack, 0 );
+				if( raven_client ){
+					raven_client.captureError( err, {
+						extra: {
+							context: context
+						}
+					});
+				}
 				if( callback ) return callback( context );
 			}
 			if( callback ) return callback( preprocessed_context );

lib/server.js

@@ -4,6 +4,8 @@ const DEFAULT_PORT = 8080;
 const DEFAULT_LIVERELOAD_PORT = 35729;
 const DEFAULT_DEV_ASSETS_MAX_AGE = 0;
 const DEFAULT_PROD_ASSETS_MAX_AGE = 1000 * 60 * 60 * 24 * 365; // 1 year, in ms
+const SENTRY_DSN = process.env.SENTRY_DSN;
+const SENTRY_LEVEL = process.env.SENTRY_LEVEL;
 
 // native
 var fs = require('fs');
@@ -26,6 +28,7 @@ handlebars_helper.help( Handlebars );
 var express_handlebars = require('express3-handlebars');
 var chokidar = require('chokidar');
 var glob = require('glob');
+var raven = require('raven');
 
 // our party! \o/
 var Page = require('./page.js');
@@ -99,6 +102,15 @@ var SolidusServer = function( options ){
 	}));
 	// catch-all middleware at the end of the stack for 404 handling
 	router.use( router.router );
+	// log express errors to sentry
+	// log uncaught exceptions to sentry and exit
+	if( SENTRY_DSN ){
+		router.use( raven.middleware.express( SENTRY_DSN ) );
+		var raven_client = solidus_server.raven_client = new raven.Client( SENTRY_DSN );
+		raven_client.patchGlobal( function(){
+			process.exit(1);
+		});
+	}
 	router.use( function( req, res, next ){
 		res.status( 404 );
 		// it's cheaper to check this than to fs.exists the notfound path

package.json

@@ -34,7 +34,8 @@
     "glob": "~3.2.6",
     "handlebars": "~1.0.12",
     "handlebars-helper": "~0.0.9",
-    "worker-farm": "~0.3.1"
+    "worker-farm": "~0.3.1",
+    "raven": "^0.6.2"
   },
   "devDependencies": {
     "mocha": "~1.9.0",