remonte les erreurs dans Sentry lorsqu'un appel webhook est en erreur #9573
Conversation
app/jobs/web_hook_job.rb
Outdated
| procedure = Procedure.find(procedure_id) | ||
| Typhoeus.post(procedure.web_hook_url, body: body, timeout: TIMEOUT) | ||
|
|
||
| Sentry.configure_scope do |scope| |
There was a problem hiding this comment.
Tu n'as pas besoin de faire Sentry.configure_scope. Les jobs sont enveloppés dans un scope isolé par défaut. Donc tu peux faire directement Sentry.set_tags(dossier: dossier_id, procedure: procedure_id).
There was a problem hiding this comment.
Ok, merci pour l'info :) Pour ma curiosité, je vois bien un Sentry.configure_scope dans ApplicationController mais je ne vois pas d'équivalent côté jobs. Le scope isolé par défaut par les jobs est setté où ?
There was a problem hiding this comment.
J'ai amendé le commit pour intégrer ta suggestion : c3dfd73
There was a problem hiding this comment.
J'ai amendé le commit pour intégrer ta suggestion : c3dfd73
krichtof
force-pushed
the
add-sentry-to-webhook-call
branch
from
21df9f3 to
c3dfd73
Compare
app/jobs/web_hook_job.rb
Outdated
|
|
||
| response = Typhoeus.post(procedure.web_hook_url, body: body, timeout: TIMEOUT) | ||
|
|
||
| raise StandardError, response if !response.success? |
There was a problem hiding this comment.
Ça va faire des retry en pagaille, non ? C'est ce que tu veux ? Pour moi c'est un breaking change.
There was a problem hiding this comment.
Ce n'est pas forcément ce que je veux. Ce que je souhaite, c'est avoir une trace quand la réponse du webhook n'est pas un succès.
J'ai amendé le commit en ne levant pas d'exception, mais en appelant Sentry.capture_message pour remonter l'info dans Sentry
krichtof
force-pushed
the
add-sentry-to-webhook-call
branch
from
c3dfd73 to
72d28fd
Compare
krichtof
force-pushed
the
add-sentry-to-webhook-call
branch
from
72d28fd to
3fdc3a1
Compare
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
No description provided.