Bug #28773
crash sur paramètre offset invalide
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
08 décembre 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Genre status=done&limit=1000&offset=NaN&orig=fluxfroids&...
Exception: type = '<type 'exceptions.ValueError'>', value = 'invalid literal for int() with base 10: 'NaN'' Stack trace (most recent call first): File "/usr/lib/python2.7/dist-packages/wcs/api.py", line 211, in _q_index 209 limit = int(get_request().form.get('limit', 210 get_publisher().get_site_option('default-page-size') or 20)) > 211 offset = int(get_request().form.get('offset', 0)) 212 order_by = get_request().form.get('order_by', 213 get_publisher().get_site_option('default-sort-order') or '-receipt_time')
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-api-check-limit-offset-parameters-are-valid-28773.patch 0001-api-check-limit-offset-parameters-are-valid-28773.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 5 ans
Frédéric Péters a écrit :
Ou lever un BadRequest ?
J'allais le dire.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-api-check-limit-offset-parameters-are-valid-28773.patch 0001-api-check-limit-offset-parameters-are-valid-28773.patch ajouté
Avec pour cette occasion des tests, ce qui m'a aussi rappelé que wcs/api.py passait la main à backoffice/management.py pour un appel.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Le Lucky Luke du patch.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5b4dbc5ae8ffb203910155a78ac8c4f885c82ccd Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Dec 8 08:25:24 2018 +0100 api: check limit/offset parameters are valid (#28773)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: check limit/offset parameters are valid (#28773)