Projet

Général

Profil

Bug #28773

crash sur paramètre offset invalide

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

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

Révision 5b4dbc5a (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

api: check limit/offset parameters are valid (#28773)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 5 ans

#2

Mis à jour par Frédéric Péters il y a plus de 5 ans

Ou lever un BadRequest ?

#3

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.

#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

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.

#5

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.

#6

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)
#7

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

Formats disponibles : Atom PDF