Merge pull request #327 from dotkom/fix/raven-release

Allow Sentry to track releases
dotkom · Mar 5, 2018 · 35dc1da · 35dc1da
2 parents 4d20161 + a3dafe5
commit 35dc1da
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/server/channels/connection.js b/server/channels/connection.js
@@ -28,9 +28,9 @@ const {
 } = require('../../common/actionTypes/voting');
 const { RECEIVE_USER_LIST: USER_LIST } = require('../../common/actionTypes/users');
 const { VOTING_FINISHED } = require('../../common/actionTypes/issues');
+const { getGitSha } = require('../utils/git');
 
-// eslint-disable-next-line global-require
-const APP_VERSION = require('child_process').execSync('git rev-parse HEAD').toString().trim();
+const APP_VERSION = getGitSha();
 
 const emitUserData = async (socket) => {
   emit(socket, VERSION, { version: APP_VERSION });

diff --git a/server/server.js b/server/server.js
@@ -3,6 +3,7 @@ const appConfig = require('./app');
 const channels = require('./channels/index');
 const logger = require('./logging');
 const Raven = require('raven');
+const { getGitSha } = require('./utils/git');
 
 (async () => {
   const app = await appConfig();
@@ -18,6 +19,7 @@ const Raven = require('raven');
     tags: {
       app: 'backend',
     },
+    release: getGitSha(),
   })
   .install();
 

diff --git a/server/utils/git.js b/server/utils/git.js
@@ -0,0 +1,9 @@
+const childProcess = require('child_process');
+
+function getGitSha() {
+  return childProcess.execSync('git rev-parse HEAD').toString().trim();
+}
+
+module.exports = {
+  getGitSha,
+};