search

Grafikart / Grafikart.fr

Dépôt pour la nouvelle version de Grafikart.fr
642 stars 180 forks source link

413 Request Entity Too Large #468

Open abourtnik opened 1 year ago

abourtnik commented 1 year ago

Ce n'est pas un bug critique.

Mais si on upload un ficher trop lourd au niveau de l'avatar, on tombe sur une page d'erreur par default de nginx "413 Request Entity Too Large".

Ca serait peut être bien d'afficher un message plus parlant a l'utilisateur finale.

Capture d’écran 2023-02-07 à 15 33 25

bernard-ng commented 6 months ago

@abourtnik le fichier que tu as uploaded avait quel taille ? Histoire de poser une limite dans les vérifications

abourtnik commented 6 months ago

J'ai essayé avec un fichier vidéo de 50 Mo. Il faudrait ajouter une restriction pour les fichiers images uniquement à l'input. Et de verifier la taille et l'extensions du fichier en JS avant de le passer au back end.

bernard-ng commented 6 months ago

Je crois qu'il faudrait également limiter la taille dans le Backend (@Grafikart c'est assez critique).

https://github.com/Grafikart/Grafikart.fr/blob/6e9e963b820b3f2fedeb8195d9b80cea5c75949e/src/Domain/Profile/Dto/AvatarDto.php#L13-L15

Ensuite, restreindre le fichier uploadable dans le frontend peut être :

https://github.com/Grafikart/Grafikart.fr/blob/6e9e963b820b3f2fedeb8195d9b80cea5c75949e/templates/account/_header.html.twig#L4-L11

enfin rajouter une restriction pour les inputs type file, qui accept les images :

https://github.com/Grafikart/Grafikart.fr/blob/6e9e963b820b3f2fedeb8195d9b80cea5c75949e/assets/elements/AutoSubmit.js#L4-L6

Ta pull request serait bienvenue @abourtnik, Je t'assiterai