Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[QA - Sentry] Ajouter un tag notify_usager pour les erreurs de mail #3169

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Oct 11, 2024

Ticket

#3168

Description

Afin d'être plus efficace dans la recherche de mail en cas de problème sur les envoi de mail, ajouter un tag sur l'erreur afin de trouver rapidement les mails important qui aurait du être envoyé à l'usager

Changements apportés

  • Catcher toutes les erreurs (Throwable à la place d'Exception)
  • Dans le traitement des exceptions ajouter les tags mailer_type et notify_usager

Pré-requis

  • Arrêter le serveur de mail docker compose stop histologe_mailer
  • MAJ .env
SENTRY_DSN=(voir scalingo)
SENTRY_ENVIRONMENT=dev
SENTRY_TRACES_SAMPLE_RATE=1.0
  • Passer l'application en mode production
    composer dump-env prod

Tests

  • Déposer et accepter un signalement
  • Envoyer un suivi usager
  • Générer des mails de visite
  • Faire une recherche sentry notify_usager:yes
    image
  • Combiner une recherche sentry
    image
  • TNR - Redémarrer le serveur de mail de mail et vérfier que tout est OK

Copy link

sonarcloud bot commented Oct 11, 2024

@sfinx13 sfinx13 requested review from hmeneuvrier, numew and emilschn and removed request for hmeneuvrier and numew October 11, 2024 09:48
Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je n'ai pas réussi à tester, parce que quand je dump l'env en prod, ça ne marche plus en local, et je ne comprends pas pourquoi.
Par contre, ma relecture est OK, et mes tests sur sentru (sur tes erreurs) sont ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants