diff --git a/api/src/controllers/organisation.js b/api/src/controllers/organisation.js index 2db267533..bdcdc9da6 100644 --- 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 index 4cb1d8b81..dcc55dd31 100644 --- 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 index c8bd50f14..a6725d2a0 100644 --- 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 index f4f2a6d1b..6a451e713 100644 --- 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 index 84a77115e..de55033ab 100644 --- 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 index 106b1f9d6..b7d5e867e 100644 --- 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 index e114c1254..950988df3 100644 --- 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 index 9ddf13e53..9272a3d84 100644 --- 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 index a5c280411..8b9f92cfc 100644 --- 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 index 7d3849ec5..e836e163b 100644 --- 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 index feeb70e28..ece8d7331 100644 --- 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 index cbd8a5351..d80b85b2e 100644 --- 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 index 6419f0fcb..7b04a5004 100644 --- 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 index 28467ee64..025ff8758 100644 --- 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 index 8fdadd289..1fc88b480 100644 --- 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 index de3a0a556..5c43fe2c0 100644 --- 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 index 811bf4900..65fd7ad1a 100644 --- 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
{(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 (