Project

General

Profile

Anomalie #4347

Ne pas logger lorsque l’on propage une exception

Added by Daniel Dehennin over 7 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
developpeurs_eole
Category:
-
Start date:
10/26/2012
Due date:
05/31/2013
% Done:

100%

Spent time:
Distribution:
EOLE 2.4

Description

Les bibliothèques ne doivent pas logger si elles propagent les exceptions, par exemple, le code suivant est erroné :

import traceback
try:
    somecode()
except Exception, e:
    log.debug(traceback.format_exc())
    raise e

ou encore :

if fail:
    msg="Something is going wrong" 
    log.error(msg)
    raise Exception(msg)

Cela conduit à un double log des erreurs comme dans source:creole/template.py?rev=85dbcef#L283

root@eolebase:~# CreoleCat -t interfaces
Erreur lors de l'instanciation du template /var/lib/creole/interfaces : [Errno 2] No such file or directory: '/etc/eole/containers_bridge.conf'
Erreur lors de l'instanciation du template /var/lib/creole/interfaces : [Errno 2] No such file or directory: '/etc/eole/containers_bridge.conf'

Related issues

Related to python-pyeole - Evolution #4253: Revoir la bibliothèque d’aide au système de logging Fermé 10/10/2012
Related to creole - Anomalie #4830: Traceback à l'étape "Installation des paquets" en cas de paquet non installable Fermé 03/11/2013 03/15/2013

Associated revisions

Revision 286a4fbe (diff)
Added by Daniel Dehennin over 7 years ago

Ne pas logger les messages des exceptions que l’on propage

  • creole/template.py (CreoleTemplateEngine.process): Suppression des log
    dans les gestionnaires d’exceptions qui propage le message d’exception.

Ref: #4347 @2m

Revision 74552bd6 (diff)
Added by Joël Cuissinat almost 7 years ago

  • pyeole/pkg.py : pas de gestion d'exception de bas niveau

Fixes #4830 @20m
Ref #4347

History

#1 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Nouveau to En attente d'informations
  • Target version changed from Eole 2.4-dev-1 to Eole 2.4-dev-2

#2 Updated by Joël Cuissinat almost 7 years ago

  • Target version changed from Eole 2.4-dev-2 to Eole 2.4-dev-3
  • % Done changed from 0 to 30

#3 Updated by Daniel Dehennin over 6 years ago

  • Due date set to 05/31/2013
  • Status changed from En attente d'informations to Résolu
  • % Done changed from 30 to 100

Je ne vois pas d’autre log mal placés dans creole.

#4 Updated by Joël Cuissinat over 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF