Development #24591
traces par connecteur
0%
Description
Objectif : sur une instance de connecteur donnée, avoir un "debug_email" qui indique le mail du développeur/mainteneur à qui envoyer les traces.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 6 ans
J'ai imaginé faire ce ticket puis je me suis dit que dans w.c.s. on n'offrait pas de possibilité de notification par email des erreurs, qu'on pouvait réfléchir pareil pour passerelle et travailler sur les logs (d'où #24588 et le rappel d'anciens).
Mis à jour par Thomas Noël il y a presque 6 ans
- faire un PasserelleEmailHandler héritier de django.utils.log.AdminEmailHandler
- le init reçoit record.request donc l'URL en jeu
- surcharger le init : si cette URL correspond à une BaseResource, alors chercher le instance.debug_email correspondant
- surcharger le send_email, s'il y a un debug_email envoyer à cette adresse (sinon super(), ie aux admins)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Je me dis que ceinture et bretelle c'est pas si mal; et puis si on spam un responsable quelque part on aura peut-être une chance que les choses soient réparées des fois (je pense au connecteur hébé à Montpellier par exemple, si on avait spammé la DSI de la métropole de traces peut-être que le truc aurait été stable plus tôt).
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Duplique Development #23575: Avoir un système de notification par mail des tiers ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
Sans passer par des handlers etc., très basiquement, avant de partir dans #27162, j'étais là-dessus, en bas de ProxyLogger::_log :
admins = settings.ADMINS if self.connector.trace_emails: admins = [(x, x) for x in self.connector.trace_emails.splitlines()] with override_settings(ADMINS=admins): getattr(self._logger, levelname.lower())(message, *args, **kwargs)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-general-add-possibility-to-override-trace-emails-rec.patch 0001-general-add-possibility-to-override-trace-emails-rec.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En repartant de cette piste semaine dernière, le patch.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0002-tests-add-test-on-trace_email-merge-with-previous-pa.patch 0002-tests-add-test-on-trace_email-merge-with-previous-pa.patch ajouté
- Fichier 0001-general-add-possibility-to-override-trace-emails-rec.patch 0001-general-add-possibility-to-override-trace-emails-rec.patch ajouté
J'ai ajouté un test, c'est tout bon.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Super,
commit d1fbd69e098b295d97ba7d7e1f0960ebb86bdb49 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Nov 25 15:08:19 2018 +0100 general: add possibility to override trace emails recipients (#24591)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
general: add possibility to override trace emails recipients (#24591)