fix: retire les données potentiellement sensibles de sentry

SocialGouv · Feb 8, 2024 · 08ff510 · 08ff510
1 parent 86499b3
commit 08ff510
Show file tree

Hide file tree

Showing 38 changed files with 109 additions and 138 deletions.
diff --git a/api/src/controllers/organisation.js b/api/src/controllers/organisation.js
@@ -27,7 +27,6 @@ const {
 const mailservice = require("../utils/mailservice");
 const validateUser = require("../middleware/validateUser");
 const { looseUuidRegex, customFieldSchema, positiveIntegerRegex } = require("../utils");
-const { capture } = require("../sentry");
 const { serializeOrganisation } = require("../utils/data-serializer");
 const { defaultSocialCustomFields, defaultMedicalCustomFields } = require("../utils/custom-fields/person");
 const { mailBienvenueHtml } = require("../utils/mail-bienvenue");

diff --git a/api/src/utils/mailservice.js b/api/src/utils/mailservice.js
@@ -33,7 +33,7 @@ const sendEmail = async (address, subject, text, html) => {
     }),
   });
   if (!emailSentResponse.ok) {
-    capture(new Error("Email not sent"), { extra: { address, subject, text, response: emailSentResponse } });
+    capture(new Error("Email not sent"), { extra: { address, subject, response: emailSentResponse } });
   }
   return emailSentResponse;
 };

diff --git a/app/src/recoil/actions.js b/app/src/recoil/actions.js
@@ -76,7 +76,7 @@ export const prepareActionForEncryption = (action) => {
       "L'action n'a pas été sauvegardée car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { action } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/comments.js b/app/src/recoil/comments.js
@@ -28,7 +28,7 @@ export const prepareCommentForEncryption = (comment) => {
       "Le commentaire n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { comment } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/consultations.js b/app/src/recoil/consultations.js
@@ -76,7 +76,7 @@ export const prepareConsultationForEncryption = (customFieldsConsultations) => (
       "La consultation n'a pas été sauvegardée car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { consultation } });
+    capture(error);
     throw error;
   }
   const consultationTypeCustomFields = customFieldsConsultations.find((consult) => consult.name === consultation.type)?.fields || [];

diff --git a/app/src/recoil/medicalFiles.js b/app/src/recoil/medicalFiles.js
@@ -30,7 +30,7 @@ export const prepareMedicalFileForEncryption = (customFieldsMedicalFile) => (med
       "Le dossier médical n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { medicalFile } });
+    capture(error);
     throw error;
   }
   const encryptedFieldsIncludingCustom = [...customFieldsMedicalFile.map((f) => f.name), ...encryptedFields];

diff --git a/app/src/recoil/passages.js b/app/src/recoil/passages.js
@@ -29,7 +29,7 @@ export const preparePassageForEncryption = (passage) => {
       "Le passage n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { passage } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/persons.js b/app/src/recoil/persons.js
@@ -116,7 +116,7 @@ export const usePreparePersonForEncryption = () => {
         "La personne n'a pas été sauvegardée car son format était incorrect.",
         "Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { person } });
+      capture(error);
       throw error;
     }
     const encryptedFields = personFields.filter((f) => f.encrypted).map((f) => f.name);

diff --git a/app/src/recoil/places.js b/app/src/recoil/places.js
@@ -25,7 +25,7 @@ export const preparePlaceForEncryption = (place) => {
       "Le lieu n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { place } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/relPersonPlace.js b/app/src/recoil/relPersonPlace.js
@@ -28,7 +28,7 @@ export const prepareRelPersonPlaceForEncryption = (relPersonPlace) => {
       "La relation entre le lieu et la personne n'a pas été sauvegardée car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { relPersonPlace } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/rencontres.js b/app/src/recoil/rencontres.js
@@ -31,7 +31,7 @@ export const prepareRencontreForEncryption = (rencontre) => {
       "La rencontre n'a pas été sauvegardée car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { rencontre } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/reports.js b/app/src/recoil/reports.js
@@ -43,7 +43,7 @@ export const prepareReportForEncryption = (report) => {
       "Le compte-rendu n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { report } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/territory.js b/app/src/recoil/territory.js
@@ -25,7 +25,7 @@ export const prepareTerritoryForEncryption = (territory) => {
       "Le territoire n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { territory } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/recoil/territoryObservations.js b/app/src/recoil/territoryObservations.js
@@ -101,7 +101,7 @@ export const prepareObsForEncryption = (customFields) => (obs) => {
       "L'observation n'a pas été sauvegardée car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { obs } });
+    capture(error);
     throw error;
   }
   const encryptedFields = [...customFields.map((f) => f.name), ...compulsoryEncryptedFields];

diff --git a/app/src/recoil/treatments.js b/app/src/recoil/treatments.js
@@ -33,7 +33,7 @@ export const prepareTreatmentForEncryption = (treatment) => {
       "Le traitement n'a pas été sauvegardé car son format était incorrect.",
       "Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
     );
-    capture(error, { extra: { treatment } });
+    capture(error);
     throw error;
   }
   const decrypted = {};

diff --git a/app/src/scenes/Actions/Action.js b/app/src/scenes/Actions/Action.js
@@ -232,7 +232,7 @@ const Action = ({ navigation, route }) => {
       if (!!newAction.completedAt) await createReportAtDateIfNotExist(newAction.completedAt);
       return response;
     } catch (error) {
-      capture(error, { extra: { message: 'error in updating action', action } });
+      capture(error, { extra: { message: 'error in updating action' } });
       return { ok: false, error: error.message };
     }
   };

diff --git a/dashboard/src/components/CommentsGeneric.js b/dashboard/src/components/CommentsGeneric.js
@@ -13,7 +13,6 @@ import { FullScreenIcon } from '../assets/icons/FullScreenIcon';
 import DatePicker from './DatePicker';
 import { outOfBoundariesDate } from '../services/date';
 import AutoResizeTextarea from './AutoresizeTextArea';
-import { capture } from '../services/sentry';
 import UserName from './UserName';
 import CustomFieldDisplay from './CustomFieldDisplay';
 
@@ -229,14 +228,6 @@ function CommentsTable({ comments, onDisplayComment, onEditComment, onAddComment
         <tbody className="small">
           {(comments || []).map((comment, i) => {
             if (!comment.type) throw new Error('type is required');
-            if (comment.type === 'person' && !comment.person) {
-              capture(new Error('person is required'), { extra: { comment } });
-              return null;
-            }
-            if (comment.type === 'action' && !comment.action) {
-              capture(new Error('action is required'), { extra: { comment } });
-              return null;
-            }
             return (
               <tr key={comment._id} className={[`tw-bg-${color} tw-w-full`, i % 2 ? 'tw-bg-opacity-0' : 'tw-bg-opacity-5'].join(' ')}>
                 <td
@@ -298,38 +289,34 @@ function CommentsTable({ comments, onDisplayComment, onEditComment, onAddComment
                           className="tw-ml-auto tw-block"
                           onClick={(e) => {
                             e.stopPropagation();
-                            try {
-                              const searchParams = new URLSearchParams(location.search);
-                              switch (comment.type) {
-                                case 'action':
-                                  searchParams.set('actionId', comment.action);
-                                  history.push(`?${searchParams.toString()}`);
-                                  break;
-                                case 'person':
-                                  history.push(`/person/${comment.person}`);
-                                  break;
-                                case 'passage':
-                                  history.push(`/person/${comment.person}?passageId=${comment.passage}`);
-                                  break;
-                                case 'rencontre':
-                                  history.push(`/person/${comment.person}?rencontreId=${comment.rencontre}`);
-                                  break;
-                                case 'consultation':
-                                  searchParams.set('consultationId', comment.consultation._id);
-                                  history.push(`?${searchParams.toString()}`);
-                                  break;
-                                case 'treatment':
-                                  searchParams.set('treatmentId', comment.treatment._id);
-                                  history.push(`?${searchParams.toString()}`);
-                                  break;
-                                case 'medical-file':
-                                  history.push(`/person/${comment.person}?tab=Dossier+Médical`);
-                                  break;
-                                default:
-                                  break;
-                              }
-                            } catch (errorLoadingComment) {
-                              capture(errorLoadingComment, { extra: { message: 'error loading comment tag button', comment } });
+                            const searchParams = new URLSearchParams(location.search);
+                            switch (comment.type) {
+                              case 'action':
+                                searchParams.set('actionId', comment.action);
+                                history.push(`?${searchParams.toString()}`);
+                                break;
+                              case 'person':
+                                history.push(`/person/${comment.person}`);
+                                break;
+                              case 'passage':
+                                history.push(`/person/${comment.person}?passageId=${comment.passage}`);
+                                break;
+                              case 'rencontre':
+                                history.push(`/person/${comment.person}?rencontreId=${comment.rencontre}`);
+                                break;
+                              case 'consultation':
+                                searchParams.set('consultationId', comment.consultation._id);
+                                history.push(`?${searchParams.toString()}`);
+                                break;
+                              case 'treatment':
+                                searchParams.set('treatmentId', comment.treatment._id);
+                                history.push(`?${searchParams.toString()}`);
+                                break;
+                              case 'medical-file':
+                                history.push(`/person/${comment.person}?tab=Dossier+Médical`);
+                                break;
+                              default:
+                                break;
                             }
                           }}>
                           <div className="tw-rounded tw-border tw-border-blue-900 tw-bg-blue-900/10 tw-px-1">

diff --git a/dashboard/src/components/DocumentsGeneric.tsx b/dashboard/src/components/DocumentsGeneric.tsx
@@ -469,7 +469,7 @@ function AddDocumentInput({ personId, onAddDocuments }: AddDocumentInputProps) {
             file: fileToUpload,
           });
           if (!docResponse.ok || !docResponse.data) {
-            capture('Error uploading document', { extra: { docResponse } });
+            capture('Error uploading document', { extra: { docResponseError: docResponse.error } });
             toast.error(`Une erreur est survenue lors de l'envoi du document ${fileToUpload?.filename}`);
             return;
           }

diff --git a/dashboard/src/recoil/actions.js b/dashboard/src/recoil/actions.js
@@ -83,7 +83,7 @@ export const prepareActionForEncryption = (action, { checkRequiredFields = true
       toast.error(
         "L'action n'a pas été sauvegardée car son format était incorrect. Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { action } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/comments.js b/dashboard/src/recoil/comments.js
@@ -35,7 +35,7 @@ export const prepareCommentForEncryption = (comment, { checkRequiredFields = tru
       toast.error(
         "Le commentaire n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { comment } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/consultations.ts b/dashboard/src/recoil/consultations.ts
@@ -82,7 +82,7 @@ export const prepareConsultationForEncryption =
         toast.error(
           "La consultation n'a pas été sauvegardée car son format était incorrect. Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
         );
-        capture(error, { extra: { consultation } });
+        capture(error);
         throw error;
       }
     }

diff --git a/dashboard/src/recoil/medicalFiles.ts b/dashboard/src/recoil/medicalFiles.ts
@@ -35,7 +35,7 @@ export const prepareMedicalFileForEncryption =
         toast.error(
           "Le dossier médical n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
         );
-        capture(error, { extra: { medicalFile } });
+        capture(error);
         throw error;
       }
     }

diff --git a/dashboard/src/recoil/passages.js b/dashboard/src/recoil/passages.js
@@ -36,7 +36,7 @@ export const preparePassageForEncryption = (passage, { checkRequiredFields = tru
       toast.error(
         "Le passage n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { passage } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/persons.ts b/dashboard/src/recoil/persons.ts
@@ -149,7 +149,7 @@ export const usePreparePersonForEncryption = () => {
         toast.error(
           "La personne n'a pas été sauvegardée car son format était incorrect. Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
         );
-        capture(error, { extra: { person } });
+        capture(error);
         throw error;
       }
     }

diff --git a/dashboard/src/recoil/places.js b/dashboard/src/recoil/places.js
@@ -32,7 +32,7 @@ export const preparePlaceForEncryption = (place, { checkRequiredFields = true }
       toast.error(
         "Le lieu n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { place } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/relPersonPlace.js b/dashboard/src/recoil/relPersonPlace.js
@@ -35,7 +35,7 @@ export const prepareRelPersonPlaceForEncryption = (relPersonPlace, { checkRequir
       toast.error(
         "Le lieu n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { relPersonPlace } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/rencontres.js b/dashboard/src/recoil/rencontres.js
@@ -38,7 +38,7 @@ export const prepareRencontreForEncryption = (rencontre, { checkRequiredFields =
       toast.error(
         "La rencontre n'a pas été sauvegardée car son format était incorrect. Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { rencontre } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/reports.js b/dashboard/src/recoil/reports.js
@@ -84,7 +84,7 @@ export const prepareReportForEncryption = (report, { checkRequiredFields = true
       toast.error(
         "Le compte-rendu n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { report } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/territory.js b/dashboard/src/recoil/territory.js
@@ -32,7 +32,7 @@ export const prepareTerritoryForEncryption = (territory, { checkRequiredFields =
       toast.error(
         "Le territoire n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { territory } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/recoil/territoryObservations.js b/dashboard/src/recoil/territoryObservations.js
@@ -110,7 +110,7 @@ export const prepareObsForEncryption =
         toast.error(
           "L'observation n'a pas été sauvegardée car son format était incorrect. Vous pouvez vérifier son contenu et tenter de la sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
         );
-        capture(error, { extra: { obs } });
+        capture(error);
         throw error;
       }
     }

diff --git a/dashboard/src/recoil/treatments.ts b/dashboard/src/recoil/treatments.ts
@@ -47,7 +47,7 @@ export const prepareTreatmentForEncryption = (treatment: TreatmentInstance, { ch
       toast.error(
         "Le traitement n'a pas été sauvegardé car son format était incorrect. Vous pouvez vérifier son contenu et tenter de le sauvegarder à nouveau. L'équipe technique a été prévenue et va travailler sur un correctif."
       );
-      capture(error, { extra: { treatment } });
+      capture(error);
       throw error;
     }
   }

diff --git a/dashboard/src/scenes/person/components/PersonDocuments.tsx b/dashboard/src/scenes/person/components/PersonDocuments.tsx
@@ -153,7 +153,7 @@ const PersonDocuments = ({ person }: PersonDocumentsProps) => {
           toast.success(documentOrFolder.type === 'document' ? 'Document mis à jour' : 'Dossier mis à jour');
         } else {
           toast.error('Erreur lors de la mise à jour du document, vous pouvez contactez le support');
-          capture('Error while updating treatment document', { _person, document });
+          capture('Error while updating treatment document', { personResponseError: personResponse.error });
         }
       }}
       onAddDocuments={async (newDocuments) => {