From 8536006416cf36f816b5738533e41c70d5b7ff9d Mon Sep 17 00:00:00 2001 From: Ray Schamp Date: Thu, 12 Nov 2015 17:00:19 -0500 Subject: [PATCH] Split CLIENT_ and NODE_ SENTRY_DSN vars --- server/config.js | 2 +- server/index.js | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/server/config.js b/server/config.js index b166764a9..c2772d910 100644 --- a/server/config.js +++ b/server/config.js @@ -19,7 +19,7 @@ module.exports = { ga_tracker: process.env.GA_TRACKER || '', // Error handling - sentry_dsn: process.env.SENTRY_DSN || '', + sentry_dsn: process.env.CLIENT_SENTRY_DSN || '', // Use minified JS libraries min: (process.env.NODE_ENV === 'production') ? '.min' : '' diff --git a/server/index.js b/server/index.js index 4dad22595..b6db0d80f 100644 --- a/server/index.js +++ b/server/index.js @@ -47,20 +47,20 @@ for (var routeId in routes) { app.get(route.pattern, handler(route)); } -if (typeof process.env.SENTRY_DSN === 'string') { +if (typeof process.env.NODE_SENTRY_DSN === 'string') { var raven = require('raven'); app.get('/sentrythrow', function mainHandler () { throw new Error('Sentry Test'); }); // These handlers must be applied _AFTER_ other routes have been applied - app.use(raven.middleware.express.requestHandler(process.env.SENTRY_DSN)); - app.use(raven.middleware.express.errorHandler(process.env.SENTRY_DSN)); + app.use(raven.middleware.express.requestHandler(process.env.NODE_SENTRY_DSN)); + app.use(raven.middleware.express.errorHandler(process.env.NODE_SENTRY_DSN)); app.use(function errorHandler (err, req, res, next) { res.append('X-Sentry-ID:' + res.sentry); res.status(500); next(err); }); - raven.patchGlobal(process.env.SENTRY_DSN, function () { + raven.patchGlobal(process.env.NODE_SENTRY_DSN, function () { process.exit(-1); }); }