DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

javascript: report upload errors to Sentry

Browse source
This commit is contained in:
Pierre de La Morinerie 2019-10-24 15:54:08 +02:00
parent a6d6d72af5
commit a3057afc22
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/javascript/shared/activestorage/ujs.js
View file

@ -1,4 +1,5 @@
import ProgressBar from './progress-bar'; import ProgressBar from './progress-bar';
import { fire } from '@utils';
const INITIALIZE_EVENT = 'direct-upload:initialize'; const INITIALIZE_EVENT = 'direct-upload:initialize';
const START_EVENT = 'direct-upload:start'; const START_EVENT = 'direct-upload:start';
@ -36,8 +37,9 @@ addUploadEventListener(PROGRESS_EVENT, ({ detail: { id, progress } }) => {
ProgressBar.progress(id, progress); ProgressBar.progress(id, progress);
}); });
addUploadEventListener(ERROR_EVENT, ({ detail: { id, error } }) => { addUploadEventListener(ERROR_EVENT, event => {
ProgressBar.error(id, error); ProgressBar.error(event.detail.id, event.detail.error);
fire(document, 'sentry:capture-exception', new Error(event.detail.error));
}); });
addUploadEventListener(END_EVENT, ({ detail: { id } }) => { addUploadEventListener(END_EVENT, ({ detail: { id } }) => {

6
app/javascript/shared/track/sentry.js
View file

@ -10,4 +10,10 @@ if (enabled && key) {
scope.setUser(user); scope.setUser(user);
scope.setExtra('browser', browser.modern ? 'modern' : 'legacy'); scope.setExtra('browser', browser.modern ? 'modern' : 'legacy');
}); });
// Register a way to explicitely capture messages from a different bundle.
addEventListener('sentry:capture-exception', event => {
const error = event.detail;
Sentry.captureException(error);
});
} }