Merge pull request #1812 from France-ioi/crash-reporting-avoid-severa…

…l-dialog

Crash reporting: avoid several dialog

src/app/utils/error-handling/error-handler.ts

@@ -9,26 +9,39 @@ import { ChunkErrorService } from '../../services/chunk-error.service';
 @Injectable()
 export class AlgErrorHandler extends ErrorHandler {
 
-  private sentryErrorHandler = Sentry.createErrorHandler({
-    showDialog: true,
-  });
+  private isDialogOpen = false;
+  /**
+   * list of errors which have already been reported in this session so that we do not report the same one several times
+   */
+  private reportedErrors: string[] = [];
 
   constructor(private chunkErrorService: ChunkErrorService) {
     super();
   }
 
   handleError(err: any): void {
+    if (this.isChunkLoadingError(err)) {
+      this.chunkErrorService.emitError();
+      return;
+    }
+    const error = convertToError(err);
+    const eventId = Sentry.captureException(error);
+
+    if (!this.isDialogOpen && !this.reportedErrors.includes(error.toString())) {
+      this.isDialogOpen = true;
+      this.reportedErrors.push(error.toString());
+      Sentry.showReportDialog({ eventId, onClose: () => this.isDialogOpen = false });
+    }
+  }
+
+  isChunkLoadingError(err: any): boolean {
     const chunkErrormessages = [
       'Loading chunk [a-z_\\d]+ failed', // older ?
       'Failed to fetch dynamically imported module', // chrome
       'error loading dynamically imported module', // firefox
       'Importing a module script failed', // safari
     ];
-    if (new RegExp(chunkErrormessages.map(m => `(${m})`).join('|')).test(convertToError(err).message)) {
-      this.chunkErrorService.emitError();
-      return;
-    }
-    this.sentryErrorHandler.handleError(convertToError(err));
+    return new RegExp(chunkErrormessages.map(m => `(${m})`).join('|')).test(convertToError(err).message);
   }
 
 }