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

Merge pull request #5159 from betagouv/file-upload-really-mute-errors

Browse source
This commit is contained in:
Pierre de La Morinerie 2020-05-18 11:57:37 +02:00 committed by GitHub
commit 0738c6ff18
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -1,7 +1,8 @@
import ProgressBar from './progress-bar';
import {
errorFromDirectUploadMessage,
FAILURE_CONNECTIVITY
ERROR_CODE_READ,
FAILURE_CLIENT
} from './file-upload-error';
import { fire } from '@utils';
@ -58,8 +59,10 @@ addUploadEventListener(ERROR_EVENT, (event) => {
ProgressBar.error(id, errorMsg);
// Report unexpected client errors to Sentry.
// (But ignore usual client errors, or errors we can monitor better on the server side.)
let error = errorFromDirectUploadMessage(errorMsg);
if (error.failureReason != FAILURE_CONNECTIVITY) {
if (error.failureReason == FAILURE_CLIENT && error.code != ERROR_CODE_READ) {
fire(document, 'sentry:capture-exception', error);
}
});