Projet

Général

Profil

Development #24591

traces par connecteur

Ajouté par Thomas Noël il y a presque 6 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
15 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Duplique Passerelle - Development #23575: Avoir un système de notification par mail des tiersFermé03 mai 2018

Actions

Révisions associées

Révision d1fbd69e (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

general: add possibility to override trace emails recipients (#24591)

Historique

#1

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).

#2

Mis à jour par Thomas Noël il y a presque 6 ans

Idée de piste :
  • 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)
#3

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).

#4

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é
#5

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Nouveau à Fermé
#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Fermé à Nouveau
#7

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)
#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

En repartant de cette piste semaine dernière, le patch.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Je relis ça.

#10

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Assigné à mis à Benjamin Dauvergne
#12

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée
#13

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)
#14

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
#15

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF