Projet

Général

Profil

Bug #41449

accès url ics/ incomplète

Ajouté par Frédéric Péters il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
08 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

L'URL doit être ics/<varname date de début> (+ éventuellement /<varname date de fin).

Sur un accès incomplet, genre /backoffice/management/reservation-de-salle/ics/, aujourd'hui trace,

Exception:
  type = '<class 'IndexError'>', value = 'list index out of range'

Stack trace (most recent call first):
  File "/usr/lib/python3/dist-packages/wcs/backoffice/management.py", line 1926, in _q_traverse
  1924                     # allow trailing slash
  1925                     path = path[:-1]
> 1926                 start_date_field_varname = path[0]
  1927                 end_date_field_varname = None
  1928                 if len(path) == 2:

Fichiers

0001-ics-do-not-raise-if-ics-acess-url-do-not-provide-par.patch (3,05 ko) 0001-ics-do-not-raise-if-ics-acess-url-do-not-provide-par.patch Nicolas Roche, 09 avril 2020 15:47
0001-ics-return-404-on-uncomplete-url-access-41449.patch (2,89 ko) 0001-ics-return-404-on-uncomplete-url-access-41449.patch Nicolas Roche, 09 avril 2020 18:03
0001-ics-return-404-on-incomplete-ics-url-access-41449.patch (2,9 ko) 0001-ics-return-404-on-incomplete-ics-url-access-41449.patch Nicolas Roche, 09 avril 2020 19:28

Révisions associées

Révision 4c0b98d0 (diff)
Ajouté par Nicolas Roche il y a environ 4 ans

ics: return 404 on incomplete ics url access (#41449)

Historique

#1

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

#2

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

acess dans la description. uncomplete dans un commentaire.

Côté code je préfère avoir quitter la fonction/méthode au début, plutôt qu'ajouter un niveau d'indentation, i.e. qqch genre

+                if len(path) not in (1, 2):
+                     raise errors.TraversalError()
#4

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

Je n'ai pas été clair : uncomplete n'est pas un vrai mot, et acess non plus.

#5

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

  • Fichier 0001-ics-return-404-on-uncomplete-url-access-41449.patch ajouté
#6

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

  • Fichier 0001-ics-return-404-on-uncomplete-url-access-41449.patch supprimé
#7

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

Désolé, je n'avais pas compris.
(maintenant j'avoue que ça le fait pas trop non plus "incomplete" en anglais)

#8

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4c0b98d0a84ada7a876aadb1fcb7a46e321a757b
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Apr 9 14:59:47 2020 +0200

    ics: return 404 on incomplete ics url access (#41449)
#10

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF