Skip to content

Commit

Permalink
Name sentry stuff
Browse files Browse the repository at this point in the history
  • Loading branch information
bcomnes committed Nov 12, 2017
1 parent 727cb4f commit cf62498
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 7 deletions.
2 changes: 1 addition & 1 deletion main/index.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
var isDev = require('electron-is-dev')
if (!isDev) {
require('../sentry')()
require('../sentry')('main')
}
var { app, ipcMain } = require('electron')
var Config = require('electron-store')
Expand Down
6 changes: 5 additions & 1 deletion renderer/audio.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@
<h1>🎵</h1>
<audio id="audio"></audio>
<audio id="needle"></audio>
<script>require('./audio/index.js')</script>
<script>
var isDev = require('electron-is-dev')
if (!isDev) require('../sentry.js')('audio')
require('./audio/index.js')
</script>
</body>
</html>
2 changes: 1 addition & 1 deletion renderer/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
<main id="app"></main>
<script>
var isDev = require('electron-is-dev')
if (!isDev) require('../sentry.js')()
if (!isDev) window.raven = require('../sentry.js')('player')
if (isDev || process.env.DEV_SERVER) {
var bundle = document.createElement('script')
bundle.src = 'http://localhost:9966/bundle.js'
Expand Down
8 changes: 4 additions & 4 deletions sentry.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ var RAVEN_C = '243594'
var MAIN_THREAD = 'browser'
var RENDERER = 'renderer'

function getopts (processtype) {
function getopts (processtype, name) {
return {
captureUnhandledRejections: true,
name: 'Hyperamp',
name: name || 'Hyperamp',
release: require('./package.json').version,
extra: {
platform: os.platform(),
Expand All @@ -26,7 +26,7 @@ function getopts (processtype) {
}
}

function setup () {
function setup (name) {
// process.type === 'browser' : main thread
// process.type === 'renderer' : electron window
var raven = process.type === MAIN_THREAD ? require('raven') : require('raven-js')
Expand Down Expand Up @@ -54,7 +54,7 @@ The app may now quit - you can safely reopen it.`,
// Renderer stuff only
}

var sentry = raven.config(url, getopts(process.type)).install()
var sentry = raven.config(url, getopts(process.type), name).install()
console.log('Sentry installed')
return sentry
}
Expand Down

0 comments on commit cf62498

Please sign in to comment.