Afficher correctement les erreurs de taille (413 avec nginx au moins)
#4484
Open
opened 3 years ago by rastapopoulos
·
3 comments
Loading…
Reference in New Issue
There is no content yet.
Delete Branch '%!s(<nil>)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
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.
Version cible mise à 4.0
Problème similaire avec les 429.
Une PR pour régler ce soucis : #4848
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 ?