Projet

Général

Profil

Bug #2521

Les exceptions levés dans Publisher mais avant la définition de self.statsd provoque une exception dans finish_failed_request

Ajouté par Benjamin Dauvergne il y a environ 11 ans. Mis à jour il y a plus de 10 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Version cible:
-
Début:
19 février 2013
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Planning:

Description


2013-02-19 11:01:17 [13682] [ERROR] Error processing request.
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/gunicorn/workers/sync.py", line 71, in handle
    self.handle_request(req, client, addr)
  File "/usr/lib/pymodules/python2.7/gunicorn/workers/sync.py", line 101, in handle_request
    respiter = self.wsgi(environ, resp.start_response)
  File "/usr/lib/pymodules/python2.7/quixote/wsgi.py", line 36, in __call__
    response = self.publisher.process_request(request)
  File "/usr/lib/pymodules/python2.7/quixote/publish.py", line 281, in process_request
    output = self.finish_failed_request()
  File "/home/bdauvergne/wd/EO/wcs/wcs.git/wcs/qommon/publisher.py", line 257, in finish_failed_request
    self.statsd.increment('failed-request')
AttributeError: WcsPublisher instance has no attribute 'statsd'
Je ne sais pas trop quel est la meilleurse solution:
  • rendre conditionnel l'appel à
    increment()
    en vérifiant que
    self.statsd
    est bien défini
  • ou bien défini self.statsd plus tôt dans le cycle de vie du publisher.

Fichiers

Historique

#2

Mis à jour par Thomas Noël il y a environ 11 ans

Yep, le patch de Fred est une bonne solution. Push !

#3

Mis à jour par Frédéric Péters il y a environ 11 ans

  • Statut changé de Solution déployée à Résolu (à déployer)

Ça roule, hop.

#4

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF