413 Request Entity Too Large

[abourtnik]

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.

[bernard-ng]

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

[abourtnik]

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]

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

Grafikart.fr/src/Domain/Profile/Dto/AvatarDto.php

Lines 13 to 15 in 6e9e963

	#[Assert\Image(mimeTypes: ['image/jpeg', 'image/png'], minWidth: 110, maxHeight: 1400, maxWidth: 1400, minHeight: 110)]
	public UploadedFile $file,
	public User $user

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

Grafikart.fr/templates/account/_header.html.twig

Lines 4 to 11 in 6e9e963

	<form class="profil-header__avatar" enctype="multipart/form-data" method="post" action="{{ path('user_avatar') }}"
	is="auto-submit">
	<img src="{{ user | avatar }}" alt="">
	<div class="profil-header__upload">
	{{ icon('cloud') }}
	</div>
	<input type="file" name="avatar"/>
	</form>

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

Grafikart.fr/assets/elements/AutoSubmit.js

Lines 4 to 6 in 6e9e963

	input.addEventListener('change', () => {
	this.submit()
	})

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