Bug #26012
crash sur /okina/iti/cities
Début:
01 septembre 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
à vérifier, est-ce que Okina aurait changé un truc dans son format ?
Error occurred while processing request
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/passerelle/utils/jsonresponse.py", line 120, in api
resp = f(*args, **kwargs)
File "/usr/lib/python2.7/dist-packages/passerelle/views.py", line 345, in perform
result = self.endpoint(request, **params)
File "/usr/lib/python2.7/dist-packages/passerelle/apps/okina/models.py", line 61, in cities
city = city['cityObject']
TypeError: string indices must be integers
Request repr():
<WSGIRequest
path:/okina/iti/cities/,
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Nouveau à En cours
En fait lié à un soucis de login/pass qui fait que Okina réponds :
HTTP/1.1 401 Unauthorized
...
{
"code" : 4,
"status" : 401,
"message" : "Invalid credentials"
}
et cela n'est pas géré par le connecteur.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Fichier 0001-okina-do-not-crash-on-401-403-responses-26012.patch 0001-okina-do-not-crash-on-401-403-responses-26012.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Voilà, c'est ultra minimal, juste histoire de pas logguer via un logging.exception... Je ne change pas le code d'erreur qui reste 500 comme dans tous les autres cas de pépin sur Okina (c'est comme ça et je vais pas le changer aujourd'hui).
Mis à jour par Thomas Noël il y a plus de 5 ans
- Fichier 0001-okina-do-not-crash-on-401-403-responses-26012.patch 0001-okina-do-not-crash-on-401-403-responses-26012.patch ajouté
Et donc mon patch était plutôt n'imp, voici celui qui change "rien" en dehors de pas logger comme une erreur (un raise APIError ça fait un logger.warning au lieu de logger.exception).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 847efa28a9b4db0b4b1d5d025ce8a5785ec4e167
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Mon Sep 3 15:57:52 2018 +0200
okina: do not crash on 401/403 responses (#26012)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
okina: do not crash on 401/403 responses (#26012)