Projet

Général

Profil

Bug #43223

opendatasoft, 'NoneType' object is not iterable

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
22 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Traceback:

File "/usr/lib/python2.7/dist-packages/passerelle/utils/jsonresponse.py" in api
  128.             resp = f(*args, **kwargs)

File "/usr/lib/python2.7/dist-packages/passerelle/views.py" in perform
  484.         result = self.endpoint(request, **params)

File "/usr/lib/python2.7/dist-packages/passerelle/apps/opendatasoft/models.py" in search
  79.         for record in result_response.json().get('records'):

Exception Type: TypeError at /opendatasoft/ods/search
Exception Value: 'NoneType' object is not iterable

Fichiers

0001-opendatasoft-manage-empty-content-returned-43223.patch (3,41 ko) 0001-opendatasoft-manage-empty-content-returned-43223.patch Nicolas Roche, 22 mai 2020 13:21
0001-opendatasoft-manage-error-returned-43223.patch (2,76 ko) 0001-opendatasoft-manage-error-returned-43223.patch Nicolas Roche, 22 mai 2020 16:52
0002-opendatasoft-manage-empty-contents-returned-43223.patch (5,06 ko) 0002-opendatasoft-manage-empty-contents-returned-43223.patch Nicolas Roche, 22 mai 2020 16:52

Révisions associées

Révision 7e1e0123 (diff)
Ajouté par Nicolas Roche il y a presque 4 ans

opendatasoft: manage error returned (#43223)

Historique

#1

Mis à jour par Nicolas Roche il y a presque 4 ans

#2

Mis à jour par Frédéric Péters il y a presque 4 ans

Il faut plutôt détecter la situation d'erreur, taper un 'err': 1 et les informations détaillées.

Par exemple ici le contenu de la réponse d'opendatasoft était :

{
  "error": "The query is invalid : Field 00 doesn't exist" 
}
#3

Mis à jour par Nicolas Roche il y a presque 4 ans

L'erreur a été provoqué par le message d'erreur donné ci-dessus suite à cette requête au serveur OpenDataSoft :
https://data.toulouse-metropole.fr/api/records/1.0/search/?q=2019-08-30T12%3A00%3A00+00%3A00&apikey=3723bdda42023e31f44c81f5c6f2c13a9211fd7a1e806a7489464394&dataset=01-station-meteo-meteopole
Elle est corrigée par le premier patch en exploitant le message d'erreur retourné.

Dans le second patch, je corrige les cas où cette erreur serait produite par un mauvais contenu retourné par le serveur (dites-moi s'il faut retirer cette seconde partie).

#4

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée

Il y aura peut-être des bouts de 0002 à tirer mais disons plus tard ailleurs. Le 0001 ok.

#5

Mis à jour par Nicolas Roche il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 7e1e01230d74d5476d74fd19fe091477eb5428f2
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri May 22 16:09:41 2020 +0200

    opendatasoft: manage error returned (#43223)
#6

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF