Project

General

Profile

Development #40044

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

Added by Thomas Noël about 1 month ago. Updated 27 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
20 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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

0001-handle-storage-error-on-file-upload-40044.patch View (4.53 KB) Thomas Noël, 26 Feb 2020 11:57 AM

Associated revisions

Revision 800d81cb (diff)
Added by Thomas Noël 27 days ago

handle storage error on file upload (#40044)

History

#1 Updated by Thomas Noël about 1 month ago

Voici d'abord la gestion lors des upload.

#2 Updated by Frédéric Péters 27 days ago

  • Status changed from Solution proposée to 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 Updated by Thomas Noël 27 days ago

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 Updated by Thomas Noël 27 days ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 27 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF