Development #54862
erreur su le rafraichissement de statistiques qui n'existent pas
Début:
14 juin 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Situation d'un export/import de pages avec des graphes qui référencent des visualisations qui n'existent pas,
File "/usr/lib/python3/dist-packages/combo/apps/dataviz/models.py", line 304, in <lambda>
transaction.on_commit(lambda: spooler.refresh_statistics_data(cell_pk=self.pk))
File "/usr/lib/python3/dist-packages/combo/utils/spooler.py", line 101, in refresh_statistics_data
cell.get_statistic_data(invalidate_cache=True)
File "/usr/lib/python3/dist-packages/combo/apps/dataviz/models.py", line 300, in get_statistic_data
invalidate_cache=invalidate_cache,
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 546, in get
return self.request('GET', url, **kwargs)
File "/usr/lib/python3/dist-packages/combo/utils/requests_wrapper.py", line 137, in request
response = super(Requests, self).request(method, url, **kwargs)
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 519, in request
prep = self.prepare_request(req)
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 462, in prepare_request
hooks=merge_hooks(request.hooks, self.hooks),
File "/usr/lib/python3/dist-packages/requests/models.py", line 313, in prepare
self.prepare_url(url, params)
File "/usr/lib/python3/dist-packages/requests/models.py", line 387, in prepare_url
raise MissingSchema(error)
requests.exceptions.MissingSchema: Invalid URL '': No schema supplied. Perhaps you meant http://?
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Fichier 0001-dataviz-do-not-crash-on-missing-statistic-url-54862.patch 0001-dataviz-do-not-crash-on-missing-statistic-url-54862.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pour info c'est parce qu'à l'import, on crée l'object Statistic en faisant get_or_create(slug=slug, site_slug=site_slug, service_slug=service_slug) et ce truc est censé raccrocher une visu existante ou alors ce sera le cas à la prochaine synchro.
Patch trivial, juste à signaler la ligne resp = app.get(location) # populate cache parce que ce test ne marchait que quand on le lançait avec l'ensemble mais unitairement il foirait.
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit dbc0fee3ade809f8d8bfe626ac7b0f35def44f7f
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jun 30 10:28:19 2021 +0200
dataviz: do not crash on missing statistic url (#54862)Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
dataviz: do not crash on missing statistic url (#54862)