Afficher correctement les erreurs de taille (413 avec nginx au moins) #4484

Open
opened 3 years ago by rastapopoulos · 3 comments
Owner

Quand les morceaux sont trop gros, à la fin ça fait une erreur JS dans la console sur le retour du JSON. Il faudrait que Bigup détecte ce genre de mauvais retours et afficher une erreur humaine du genre "Veuillez augmenter la taille maximum d'upload sur votre serveur".

Avec Nginx, la config par défaut est de seulement 1M, ce qui est pas beaucoup mais bon c'est comme ça c'est le défaut. Or Bigup découpe apparemment en fichier plus gros par défaut, du coup ça plante. En passant "client_max_body_size" à 8Mo tout remarche.

Quand les morceaux sont trop gros, à la fin ça fait une erreur JS dans la console sur le retour du JSON. Il faudrait que Bigup détecte ce genre de mauvais retours et afficher une erreur humaine du genre "Veuillez augmenter la taille maximum d'upload sur votre serveur". Avec Nginx, la config par défaut est de seulement 1M, ce qui est pas beaucoup mais bon c'est comme ça c'est le défaut. Or Bigup découpe apparemment en fichier plus gros par défaut, du coup ça plante. En passant "client_max_body_size" à 8Mo tout remarche.
cerdic commented 2 years ago
Owner

Version cible mise à 4.0

**Version cible mise à 4.0**
g0uZ commented 2 years ago

Problème similaire avec les 429.

Une PR pour régler ce soucis : #4848

Problème similaire avec les 429. Une PR pour régler ce soucis : https://git.spip.net/spip/bigup/pulls/4848
Poster
Owner

Du coup est-ce que #4850 a résolu ce ticket aussi ? (vu que ça dit "erreurs HTTP diverses") Ou bien faut toujours afficher une erreur mieux aux humains ?

Du coup est-ce que https://git.spip.net/spip/bigup/pulls/4850 a résolu ce ticket aussi ? (vu que ça dit "erreurs HTTP diverses") Ou bien faut toujours afficher une erreur mieux aux humains ?
b_b modified the milestone from spip-4.0 to spip-4.1 9 months ago
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: spip/bigup#4484
Loading…
There is no content yet.