Merge pull request #9580 from demarches-simplifiees/add-sentry-to-webhook-call

fix: corrige la remontée d'infos à Sentry pour les appels webhooks
This commit is contained in:
LeSim 2023-10-10 09:15:30 +00:00 committed by GitHub
commit b2962c16e5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 2 deletions

View file

@ -18,7 +18,7 @@ class WebHookJob < ApplicationJob
if !response.success?
Sentry.set_tags(procedure: procedure_id, dossier: dossier_id)
Sentry.set_extras(web_hook_url: procedure.web_hook_url)
Sentry.capture_message("Webhook error: #{response.status} // Response: #{response.body}")
Sentry.capture_message("Webhook error code: #{response.code} (#{response.return_message}) // Response: #{response.body}")
end
end
end

View file

@ -14,8 +14,11 @@ describe WebHookJob, type: :job do
context 'with error on webhook' do
it 'raises' do
allow(Sentry).to receive(:capture_message)
stub_request(:post, web_hook_url).to_return(status: 500, body: "error")
expect { job.perform_now }.to raise_error
job.perform_now
expect(Sentry).to have_received(:capture_message)
end
end
end