Rend publication_watchdog plus résistant aux exceptions

[philippemilink]

Pour éviter que le service zds-watchdog entier ne plante lorsqu'une erreur survient dans la boucle qui traite chaque demande de publication, on place le contenu de la boucle dans un bloc try/except. De cette façon, les seules erreurs qui pourraientt faire planter le service sont celles qui apparaîtraient dans le bloc except.

QA

Surtout une revue de code, s'assurer que le watchdog continue à fonctionner dans des conditions normales.

[coveralls]

Coverage decreased (-0.02%) to 87.201% when pulling c6233c2 on philippemilink:watchdog-handle-exceptions into e2c0775 on zestedesavoir:dev.

[Situphen]

Je pense qu'il serait plus lisible d'avoir un seul try avec plusieurs except comme ceci :

try:
    content = publication_event.published_object
    # ...
    publicator = PublicatorRegistry.get(publication_event.format_requested)
    publicator.publish(md_file_path, base_name)
except FailureDuringPublication:
    logger.exception("...")
    # ...
except Exception:
    logger.exception("...")
    # ...

Aussi, est-ce qu'utiliser logger.exception ne permettrait pas d'éviter d'utiliser logger.error avec e_type et e_value en paramètres ?

[philippemilink]

Je pense qu'il serait plus lisible d'avoir un seul try avec plusieurs except comme ceci :

En effet, c'est plus clair.

Aussi, est-ce qu'utiliser logger.exception ne permettrait pas d'éviter d'utiliser logger.error avec e_type et e_value en paramètres ?

Je ne connaissais pas, et c'est bien ce dont a besoin ici, comme ça on les détails de l'exception et la backtrace dans les logs.