search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

N'envoie pas à Sentry les `KeyboardInterrupt` #6678

Closed philippemilink closed 3 weeks ago

philippemilink commented 3 weeks ago

Quand on lance une commande manage.py et qu'on l'interrompt avec Ctrl+C, cela génère une exception KeyboardInterrupt qui est envoyée à Sentry, puisque c'est une exception non rattrapée par notre code. Cependant, ça n'a pas vraiment de sens d'enregistrer ça dans Sentry, puisque c'est forcément un être humain qui a provoqué consciemment et volontairement la KeyboardInterrupt. Autrement dit : ce n'est pas un vrai problème et l'utilisateur a connaissance immédiatement de l'exception ; bref, ça n'a rien à faire dans Sentry :)

Contrôle qualité

J'ai testé sur la bêta, ça fonctionne les KeyboardInterrupt ne sont plus envoyées à Sentry.

J'ai suivi cette documentation.

coveralls commented 3 weeks ago

Coverage Status

coverage: 89.205%. remained the same when pulling e75d86bc3b82a3c04687d6d95cf5ae92931404dd on philippemilink:sentry-no-log-keyboardinterrupt into f183502560aac755ab9e9ea11abe138623933298 on zestedesavoir:dev.