Bug #14224
crash moche quand quelque chose foire côté fargo
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
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-file_validation-don-t-crash-when-fargo-messed-up-142.patch 0001-file_validation-don-t-crash-when-fargo-messed-up-142.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
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.
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)
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
file_validation: don't crash when fargo messed up (#14224)