Projet

Général

Profil

Bug #45053

dashboard, traces auto-title

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é à:
Frédéric Péters
Version cible:
-
Début:
12 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Quand un moteur de recherche se trouve à tenter une URL comme /api/dashboard/auto-tile/mdr/, trace,

File "/usr/lib/python3/dist-packages/combo/apps/dashboard/views.py" in dashboard_auto_tile
  121.     request_body = json.loads(force_text(request.body))

File "/usr/lib/python3.5/json/__init__.py" in loads
  319.         return _default_decoder.decode(s)

File "/usr/lib/python3.5/json/decoder.py" in decode
  339.         obj, end = self.raw_decode(s, idx=_w(s, 0).end())

File "/usr/lib/python3.5/json/decoder.py" in raw_decode
  357.             raise JSONDecodeError("Expecting value", s, err.value) from None

Fichiers

0002-dashboard-don-t-allow-GET-request-to-autotile-45053.patch (2,1 ko) 0002-dashboard-don-t-allow-GET-request-to-autotile-45053.patch Frédéric Péters, 12 juillet 2020 09:21
0001-dashboard-raise-on-missing-parameters-in-auto-tile-4.patch (2,62 ko) 0001-dashboard-raise-on-missing-parameters-in-auto-tile-4.patch Frédéric Péters, 12 juillet 2020 09:21

Révisions associées

Révision 3e976337 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

dashboard: raise on missing parameters in auto-tile (#45053)

Révision 24980ab6 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

dashboard: don't allow GET request to autotile (#45053)

Historique

#1

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

C'est 0002 parce qu'ils font un GET, alors que seul le POST fournit un request.body, mais tant qu'à faire, 0001, pour vérifier que les paramètres fournis correspondent à la définition. (mon idée était de peut-être les permettre en query string un jour, mais pas maintenant, d'où 0002).

#2

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

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

(Les patchs sont corrects, je réalise que je peux tester le rendu de ma cellule json avec l'API dashboard_auto_tile mais j'ai l'impression que je rate quelque-chose sur l'utilisation de cette API : ma cellule est sauvée en base avec comme placeholder '_auto_tile', mais je ne comprend pas comment l'utiliser.)

#3

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 24980ab6fe0b2d4747288cf5a080f05babed65b4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Jul 11 22:56:24 2020 +0200

    dashboard: don't allow GET request to autotile (#45053)

commit 3e9763372368530f846eaf5c81371f4c2485aca4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Jul 11 22:50:21 2020 +0200

    dashboard: raise on missing parameters in auto-tile (#45053)
#4

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