Projet

Général

Profil

Development #40044

gestion des UploadStorageError lors de l'envoi d'un fichier

Ajouté par Thomas Noël il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
20 février 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le nouveau save_tempfile ajouté dans #39517 peut soulever des exceptions UploadStorageError en cas de soucis de stockage.

Il faut gérer cela aux endroits où c'est utilisé, en gros partout où il y a des session.add_tempfile :

  • root.py : tmp_upload (le point d'entrée principal pour l'upload de fichier) -- renvoyer un json (ou autre) qui déclenche de fail de jquery.fileupload.js
  • qommon/form.py : _parse de FileWithPreviewWidget (failover quand le fichier a été uploadé "normalement") -- refuser la validation
  • forms/common.py : restore_draft -- cas à étudier

Fichiers

0001-handle-storage-error-on-file-upload-40044.patch (4,53 ko) 0001-handle-storage-error-on-file-upload-40044.patch Thomas Noël, 26 février 2020 11:57

Révisions associées

Révision 800d81cb (diff)
Ajouté par Thomas Noël il y a environ 4 ans

handle storage error on file upload (#40044)

Historique

#1

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

Voici d'abord la gestion lors des upload.

#2

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

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

Tu as d'un côté "failed to store file (system error)" et de l'autre "Failed to store file (system error)", je serais pour utiliser la même chaine. (et comme "unknown storage system" n'a pas de majuscule j'irais dans la direction pas de majuscule).

#3

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

Frédéric Péters a écrit :

Tu as d'un côté "failed to store file (system error)" et de l'autre "Failed to store file (system error)", je serais pour utiliser la même chaine. (et comme "unknown storage system" n'a pas de majuscule j'irais dans la direction pas de majuscule).

C'est ce que j'avais fait au début mais à l'affichage ça fait un poil désordre :
  • failed to store file (system error) : s'affiche comme un message d'erreur de validation du champ, qui sont tous en minuscules
  • Failed to store file (system error) : s'affiche comme erreur lors de l'upload (javascript) et ces erreurs commencent par une majuscule ("Error during upload.")

Mais en réalité cette erreur ne devrait quasiment jamais apparaître (on est dans le cas d'une panne, pas d'une erreur de manip) donc j'ai tout passé en minuscule. J'ai rabasé, refait le test et je pousse dès que jenkins est vert.

#4

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 800d81cb2ef53ff2a1333331512861861a32efd5
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Feb 26 11:46:31 2020 +0100

    handle storage error on file upload (#40044)
#5

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