Development #43967
JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Début:
12 juin 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/2715/
JSONDecodeError: Expecting value: line 1 column 1 (char 0) (17 additional frame(s) were not displayed) ... File "combo/apps/lingo/models.py", line 176, in get_invoices items = requests.get(url, user=user, remote_service='auto', cache_duration=0).json() File "requests/models.py", line 897, in json return complexjson.loads(self.text, **kwargs) File "__init__.py", line 319, in loads return _default_decoder.decode(s) File "json/decoder.py", line 339, in decode obj, end = self.raw_decode(s, idx=_w(s, 0).end()) File "json/decoder.py", line 357, in raw_decode raise JSONDecodeError("Expecting value", s, err.value) from None Internal Server Error: /ajax/cell/13/lingo_activeitems-5/
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-lingo-return-empty-list-if-retrieving-invoice-fails-.patch 0001-lingo-return-empty-list-if-retrieving-invoice-fails-.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Je me rend compte à cette occasion qu'on ne log dans requests_wrapper que les status != 2xx et pas les erreurs de transport (qui ne sont pas plus gérés dans les sites d'appel). Je pense que ça milite pour une API un peu plus haut niveau au dessus de requests:
- si log_errors = True, logger aussi les erreurs de Transport avant de les remonter,
- avoir un
get/post_json()
pour gérer aussi les erreurs quand le retour n'est pas du JSON.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-lingo-report-errors-on-invoices-retrieval-43967.patch 0001-lingo-report-errors-on-invoices-retrieval-43967.patch ajouté
Changement pour rapporter à l'utilisateur que la régie est cassée pour le moment.
Mis à jour par Christophe Siraut il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0c82ae2c1845c87275dba57701152ff57d55c426 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Jun 12 10:35:26 2020 +0200 lingo: report errors on invoices retrieval (#43967)
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
lingo: report errors on invoices retrieval (#43967)