yeswiki icon indicating copy to clipboard operation
yeswiki copied to clipboard

Attach ajouter un paramètre max_image_filesize

Open MelanieMichel opened this issue 3 years ago • 7 comments

New feature / Nouvelle fonctionnalité

Description Proposition de permettre de fixer une taille max pour les images uploadées via attach permettrai de limiter l'espace disque utilisé sur des wikis avec beaucoup de contributeurs qui partagent des images

MelanieMichel avatar Sep 01 '22 08:09 MelanieMichel

@MelanieMichel c'est déjà le cas avec le paramètre "max_file_size" qu'on peut configurer dans le fichier wakka.config.php par exemple "max_file_size" => 2*1024*1024, pour restreindre à 2Mo.

On pourrait alors faire apparaître ce paramètre dans la page GererConfig. Êtes-vous d'accord @furax37 ou @gatienbataille pour cet ajout ?

@MelanieMichel tu peux dores et déjà vérifier que le paramètre mis à la main dans wakka.config.php est déjà pris en compte.

On pourrait aussi ajouter de l'aide au niveau de cette page https://yeswiki.net/?AttacH une fois la modification apportée dans GererConfig

J9rem avatar Sep 02 '22 07:09 J9rem

c'est une bonne idée de rendre accessible le max_file_size dans la gestion de la configuration, mais par contre d'avoir un tuning pour les images serait important, car on peut avoir envie d'attacher un pdf de 20mo ou une video mp4, mais tout de meme vouloir restreindre les images

mrflos avatar Sep 02 '22 07:09 mrflos

@J9rem merci pour l'info, en effet le paramètre fonctionne et il y a bien un message en cas de dépassement de la taille max.

Je rejoins @mrflos sur l'idée de différencier les images et éventuellement de donner la possibilité d'un redimensionnement coté serveur comme on le fait via les templates ? avec du coup un paramètre "reduce_uploaded_images"

MelanieMichel avatar Sep 02 '22 07:09 MelanieMichel

Est-ce que ma première proposition facile à faire de rendre disponible max_file_size dans la gestion de la configuration telle quelle est validée par @mrflos et @MelanieMichel ?

Et on garde la présente issue ouverte en attendant de trouver quelqu'un pour développer le truc plus lourd avec la gestion fine selon les types de fichiers.

Ca n'est pas très clair dans vos réponses qui sont plutôt du genre oui mais ... Je suis binaire soit c'est oui, soit c'est non avec des idées d'améliorations requises au préalable

J9rem avatar Sep 02 '22 08:09 J9rem

rendre disponible max_file_size dans la gestion de la configuration répond à la demande initiale

j'ajouterai à la documentation de attach lorsque l'option sera disponible via page configuration

J'ai ouvert une nouvelle issue à propos de la question du redimensionnement qui était un rebond ;)

MelanieMichel avatar Sep 02 '22 08:09 MelanieMichel

oui ok pour moi pour max_file dans config

gatienbataille avatar Sep 02 '22 11:09 gatienbataille

merci @MelanieMichel et @mrflos pour votre clarification ici (et par message privé). Je rajoute donc l'option max_file_size dans gérer config et on conserve la présente PR pour la deuxième partie concernant la mise en place plus tardive de pouvoir choisir une taille différente en fonction du type de fichier.

J9rem avatar Sep 02 '22 16:09 J9rem

Je ferme je pense que ce que tu as Jerem répond déjà à la plus grande partie du besoin !

seballot avatar Dec 20 '23 05:12 seballot