Projet

Général

Profil

Development #42818

taper une erreur 400 quand de la merde est tapée là où on attend un int()

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é à:
-
Version cible:
-
Début:
13 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Genre,

  type = '<class 'ValueError'>', value = 'invalid literal for int() with base 10: '&echo eradbr$()\\ rlwung\\nz^xyu||a #\' &echo eradbr$()\\rlwung\\nz^xyu||a #|" &echo eradbr$()\\ rlwung\\nz^xyu||a #''

Stack trace (most recent call first):
  File "/usr/lib/python3/dist-packages/wcs/backoffice/management.py", line 862, in listing
   860         limit = int(get_request().form.get('limit',
   861             get_publisher().get_site_option('default-page-size') or 20))
>  862         offset = int(get_request().form.get('offset', 0))
   863         order_by = get_request().form.get('order_by',
   864             get_publisher().get_site_option('default-sort-order') or '-receipt_time')

  locals:
     limit = 20
     self = <wcs.backoffice.management.ManagementDirectory object at 0x7fd675bbcac8>
     sql = <module 'wcs.sql' from '/usr/lib/python3/dist-packages/wcs/sql.py'>

Fichiers

0001-misc-raise-bad-requests-on-invalid-integers-for-limi.patch (7,6 ko) 0001-misc-raise-bad-requests-on-invalid-integers-for-limi.patch Frédéric Péters, 13 mai 2020 12:43

Révisions associées

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

misc: raise bad requests on invalid integers for limit/offset (#42818)

Historique

#1

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

#2

Mis à jour par Thomas Noël il y a presque 4 ans

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

S'il te reste des vitamines tu peux condenser dans wcs/backoffice/management.py :

        offset = None
        if 'offset' in get_request().form:
            offset = misc.get_int_or_400(get_request().form['offset'])

en

        offset = misc.get_int_or_400(get_request().form.get('offset'))

et idem pour le limit qui suit.

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

Oui j'ai été frileux là, j'ai laissé ainsi,

commit ad6be5365720bdabfc5a9dd3dd0338ed77b26a56
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed May 13 12:42:13 2020 +0200

    misc: raise bad requests on invalid integers for limit/offset (#42818)
#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