Projet

Général

Profil

Bug #6239

pas de fichier lors d'un test de max_file_size ?

Ajouté par Thomas Noël il y a plus de 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
06 janvier 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Reçu à l'instant de la recette fsb

From: auquo-test <tnoel+fsb-recette@entrouvert.com>
Subject: Quixote Traceback (AttributeError: 'NoneType' object has no attribute 'fp')

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute
'fp''

Stack trace (most recent call first):
  File "/usr/lib/python2.7/dist-packages/wcs/qommon/form.py", line 603, in _parse
   601         if self.max_file_size:
   602             # validate file size
>  603             file_size = os.path.getsize(self.value.fp.name)
   604             if file_size > self.max_file_size_bytes:
   605                 self.error = _('over file size limit (%s)') % self.max_file_size

  locals:
     token = None
     session = <BasicSession at 35d9810: faf7907e75dafbe4>
     request = <qommon.http_request.HTTPRequest instance at 0x39a1fc8>
     self = <FileWithPreviewAndStrongboxWidget at 36488d0: f5>

Fichiers

0001-form-don-t-pass-over-additional-file-checks-if-there.patch (910 octets) 0001-form-don-t-pass-over-additional-file-checks-if-there.patch Frédéric Péters, 06 janvier 2015 11:04

Historique

#1

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

Ce patch doit faire l'affaire; mais ce qu'il faudrait c'est un backport de python-webtest, pour pouvoir prendre en compte les <input type="file"/>.

(je te laisse voir si tu veux l'appliquer live sur le serveur).

#2

Mis à jour par Thomas Noël il y a plus de 9 ans

Ack pour le patch

(et déjà appliqué test et prod)

#3

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

  • Statut changé de Nouveau à Résolu (à déployer)
  • Patch proposed changé de Non à Oui

cool.

commit 822a51e9969b95436711da1d8aa833c75633e968
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jan 6 11:01:56 2015 +0100

    form: don't pass over additional file checks if there are no file (#6239)
#4

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF