Projet

Général

Profil

Bug #38362

traces emails en double quand error_email n'est pas défini

Ajouté par Christophe Siraut il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
10 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

w.c.s. permet de définir un courriel pour les tracebacks via le backoffice. Lorsque cette option n'est pas définie, la trace est envoyée 2 fois au même destinataire.

Dans wcs/qommon/logger.py :

class ApplicationLogger(DefaultLogger):
    def log_internal_error(self, error_summary, error_msg, tech_id=None):
        self.log('exception caught')
        self.error_log.write(error_msg)
        if self.error_email:
            from .emails import email
            headers = {}
            if tech_id:
                headers['References'] = '<%s@%s>' % (tech_id, os.path.basename(get_publisher().app_dir))
            email(subject='[ERROR] %s' % error_summary,
                  ....

Fichiers

0001-ApplicationLogger-do-not-send-email-when-error_email.patch (1,13 ko) 0001-ApplicationLogger-do-not-send-email-when-error_email.patch Christophe Siraut, 10 décembre 2019 17:59

Historique

#2

Mis à jour par Christophe Siraut il y a plus de 4 ans

#3

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

Je ne pense pas cela correct (ni le patch ni l'analyse).

cf ticket lié.

  • Subject: Cron <wcs@pddv-gdlyon-gnm-front-1> /usr/bin/wcs-manage cron
  • Subject: Toodego : [ERROR] [CRON] ConnectionError: error in HTTP request to passerelle.guichet-recette.grandlyon.com (("bad handshake: SysCallError(104, 'ECONNRESET')", ))

1/ ça concerne une plateforme où error_email est défini; 2/ w.c.s. envoie un message, l'autre c'est cron (qui transmet le stdout/stderr de la commande).

#4

Mis à jour par Christophe Siraut il y a plus de 4 ans

  • Statut changé de Solution proposée à Rejeté

Formats disponibles : Atom PDF