osm-fr / infrastructure

Handle tickets against osm-fr infrastructure
MIT License
21 stars 4 forks source link

permettre l'ajout de fichier h5p sur le wordpress d'osm france #449

Closed vinber closed 1 year ago

vinber commented 1 year ago

dans l'objectif de donner à voir le "mooc" d'initiation à OSM fait avec la DANE de Lyon, j'ai activé le plugin h5p de wordpress, j'ai tenté l'import de fichier h5p, cela ne fonctionne pas image

cependant la création d'une activité fonctionne donc le module marche, c'est bien l'import qui ne fonctionne pas.

bristow commented 1 year ago

Quand je teste via Librairies à téléverser, il met ce message d'erreur :

The uploaded file exceeds the upload_max_filesize directive in php.ini

Mes fichiers h5p varie entre 1.5Mo et 9.6 Mo.

à voir donc dans ce fichier. Merci !

Marc-marc-marc commented 1 year ago

valeur actuelle /etc/php/7.4/cli/php.ini:upload_max_filesize = 2M je monte a 12Mo et systemctl restart php7.4-fpm.service

vinber commented 1 year ago

je viens de tester, j'ai la même réponse Could not get posted H5P.

vinber commented 1 year ago

https://h5p.org/node/759896 (rencontre un problème "similaire")

vinber commented 1 year ago

et en tentant avec un tout petit fichier h5p de 0.5Mo cela fonctionne.

donc ce n'est pas le format mais bien en lien avec la taille ...

vinber commented 1 year ago

magnifique exemple https://www.openstreetmap.fr/test-h5p/

vinber commented 1 year ago

test sur chromium, ne fonctionne pas

jocelynj commented 1 year ago

Peut-être une limite côté nginx sur le proxy ? (me semble qu'on avait monté sur peertube, et il doit falloir faire pareil sur ce site)

vinber commented 1 year ago

est-ce que cela peut-être testé ? comment savoir la limite coté nginx ?

bristow commented 1 year ago

@cquest a vérifié le nginx lors du sotm et c'est réglé à 50 Mo. Bizarre...

m2589972 commented 1 year ago

si c'est un souci de taille d'upload, ça pourrait vous être utile de pouvoir déposer le fichier par FTP pour importer ce contenu ? je suis développeur php donc je peux essayer de faire cela.

jocelynj commented 1 year ago

Effectivement, ça pourrait être une idée d'uploader par un autre canal (je préférerais ssh que ftp ici), mais est-ce qu'on saurait l'envoyer à wordpress après ?

@vinber : tu pourrais m'envoyer le fichier que je teste, ainsi que la page d'admin à utiliser ?

bristow commented 1 year ago

J'utilise le plugin WordPress H5P sur 2 autres sites et je n'ai aucun soucis pour remonter ces fichiers. Le SSH ou FTP peuvent être une solution mais très contraignante car je ne suis pas sûr que l'on puisse récupérer facilement l'archive ensuite. Vaudrait mieux trouver ce qui bloque...

@jocelynj : Voici un fichier h5p pour tester l'upload : https://nuage03.apps.education.fr/index.php/s/dZYwYxMANWoEE3k

La page à utiliser est ici : https://www.openstreetmap.fr/wp-admin/admin.php?page=h5p_new

Merci :pray:

vinber commented 1 year ago

merci @m2589972 de la proposition. Si effectivement on peut passer par la voie pour les "humains" c'est plus simple par la suite :) mais si tu veux jouer il y a sans doute plein de choses à faire dans les issues ...

jocelynj commented 1 year ago

Je pense que le problème est ailleurs, parce que le débugguer réseau de firefox montre que le fichier .h5p n'est même pas envoyé avant l'affichage de l'erreur.

jocelynj commented 1 year ago

Ok, c'était bien un souci sur la config serveur. J'ai changé ceci sur la VM hébergeant le wordpress:

--- a/php/7.4/fpm/php.ini
+++ b/php/7.4/fpm/php.ini
@@ -691,7 +691,7 @@ auto_globals_jit = On
 ; Its value may be 0 to disable the limit. It is ignored if POST data reading
 ; is disabled through enable_post_data_reading.
 ; http://php.net/post-max-size
-post_max_size = 8M
+post_max_size = 64M

 ; Automatically add files before PHP document.
 ; http://php.net/auto-prepend-file
@@ -843,7 +843,7 @@ file_uploads = On

 ; Maximum allowed size for uploaded files.
 ; http://php.net/upload-max-filesize
-upload_max_filesize = 2M
+upload_max_filesize = 64M
jocelynj commented 1 year ago

@vinber : tu pourrais retester ?

bristow commented 1 year ago

ça fonctionne :tada: Merci @jocelynj !!

vinber commented 1 year ago

@bristow mais tu t'appelles pas @vinber :) !!!

merci à tout le monde surtout