Projet

Général

Profil

Bug #14224

crash moche quand quelque chose foire côté fargo

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 décembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'get''

Stack trace (most recent call first):
  File "/usr/lib/python2.7/dist-packages/wcs/file_validation.py", line 90, in get_document_types
    88         get_logger().warning('unable to retrieve document types from fargo')
    89         return {}
>   90     if response.get('err') == 0:
    91         result = {}
    92         for schema in response['data']:

  locals:
     response = None

  File "/usr/lib/python2.7/dist-packages/wcs/fields.py", line 841, in get_document_types
   839         document_types.update(get_cfg('filetypes', {}))
   840         # Remote documents types
>  841         document_types.update(file_validation.get_document_types())
   842         for key, document_type in document_types.iteritems():
   843             document_type['id'] = key

Parce que

def fargo_get(url):
    url = fargo_url(url)
    response, status, data, auth_header = http_get_page(url)
    if status == 200:
        return json_loads(data)
    return None

return None modifié en return {'err': 1} pour que ça passe.


Fichiers

Révisions associées

Révision f3503975 (diff)
Ajouté par Thomas Noël il y a plus de 7 ans

file_validation: don't crash when fargo messed up (#14224)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 7 ans

  • Assigné à mis à Thomas Noël
#2

Mis à jour par Thomas Noël il y a plus de 7 ans

Et donc, après avoir tenté d'ajouter aussi un peu de tests sur err==0 sans succès, ta simple proposition initiale rend bien le service.

#3

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

ack.

#4

Mis à jour par Thomas Noël il y a plus de 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit f350397597d2848d4fd06cc435d1cb540fbd2140
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Dec 7 14:27:50 2016 +0100

    file_validation: don't crash when fargo messed up (#14224)

#5

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

Formats disponibles : Atom PDF