Ensure HTTP errors in web hook logic are reported
Created by: guillett
- Lien vers Trello Sentry https://trello.com/c/sgoKsOHT/1119-remonter-les-erreurs-en-cas-de-probl%C3%A8mes-dans-les-notifications
- Des requêtes en direction du SI du Pas de Calais sont en échec. Aujourd'hui nous n'avons pas de visibilité là dessus. Avec ce dev, les notifications ayant échouées, ie celle dont la requête HTTP ne retournant pas un statut sans erreur, seront en erreur dans les delayed_jobs avec la réponse HTTP reçue pour faciliter la résolution du problème.
Dans le cas du Pas de Calais, les requêtes en erreur renvoyaient précédemment un code 200 (ie pas d'erreur). Aujourd'hui, nous recevons une 403 mais celui est pour le moment ignoré.
Je suis preneur de reformulation de l'erreur remontée.
- checklist avant review:
-
reparcourir le code rapidement pour voir les problèmes évidents (fichiers touched inutilement, debug logs qui trainent...). -
Attendre que les tests soient verts sur la CI