[Corrigé 3.2.39] Mises à jour derrière un serveur proxy

Vous avez trouvé un bug dans l'application (dernière version stable ou bêta): Décrivez le ici afin que la correction soit intégrée a la prochaine version.
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Bonjour,

Nous utilisons GestSup dans des établissements scolaires, où les serveurs sont situés derrière un serveur proxy, par lequel il faut obligatoirement passer pour télécharger en HTTP/HTTPS.

Il nous a donc fallu, en plus d'installer le paquet curl qui ne l'était pas encore dans notre cas, patcher le code actuel pour pouvoir installer les mises à jour de manière semi-automatique, via les commandes :

Code : Tout sélectionner

sed -i -E 's/curl_setopt\(\$(\S+)+, CURLOPT_SSL_VERIFYHOST, 0\);/curl_setopt\(\$\1, CURLOPT_PROXY, "http:\/\/10.0.0.1:3128"\);\ncurl_setopt\(\$\1, CURLOPT_SSL_VERIFYHOST, 0\);/g' /var/www/html/core/functions.php
sed -i -E 's/curl_setopt\(\$(\S+)+, CURLOPT_SSL_VERIFYHOST, 0\);/curl_setopt\(\$\1, CURLOPT_PROXY, "http:\/\/10.0.0.1:3128"\);\ncurl_setopt\(\$\1, CURLOPT_SSL_VERIFYHOST, 0\);/g' /var/www/html/admin/update.php
(où http://10.0.0.1:3128 est l'URL du proxy dans ce cas de figure)

Afin de rendre ces modifications plus durables et intégrées dans le code du projet, nous proposons les modifications jointes.

Bien cordialement,

Tom, pour l'équipe SambaÉdu.
Fichiers joints
gestsup.zip
(53.13 Kio) Téléchargé 71 fois
Avatar du membre
Flox
Administrateur du site
Messages : 9049
Enregistré le : jeu. 21 juin 2012 19:00

Pouvez-vous transmettre une impression écran de votre page système ?

Cdt
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.58 | MariaDB: 11.3.2 | PHP: 8.3.6 | https://doc.gestsup.fr/
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Voici le screenshot, qui prend en compte les modifications proposées (que je remets en pièces-jointe vu que j'avais manqué un curl en bas de la page system.php, et qu'il ne me semble plus possible d'éditer le premier message)

Image
Fichiers joints
Screenshot GestSup.png
Screenshot GestSup.png (308.12 Kio) Vu 890 fois
gestsup.zip
(53.1 Kio) Téléchargé 69 fois
Avatar du membre
Flox
Administrateur du site
Messages : 9049
Enregistré le : jeu. 21 juin 2012 19:00

C'est noté je regarde pour une intégration de ce paramètre pour la prochaine version.

Cdt
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.58 | MariaDB: 11.3.2 | PHP: 8.3.6 | https://doc.gestsup.fr/
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Merci !
Avatar du membre
Flox
Administrateur du site
Messages : 9049
Enregistré le : jeu. 21 juin 2012 19:00

Vous pourrez tester dans la prochaine version 3.2.37, avec ce paramètre :
2023-07-06 17_30_25_screenshot_gestsup.png
2023-07-06 17_30_25_screenshot_gestsup.png (7.63 Kio) Vu 847 fois
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.58 | MariaDB: 11.3.2 | PHP: 8.3.6 | https://doc.gestsup.fr/
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Bonjour,

Cela semble bien correspondre à notre proposition, nous testerons ça dès que la version 3.2.37 sera disponible en béta.

Bien cordialement,

Tom.
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Bonjour,

Nous avons enfin eu le temps de mettre à jour en 3.2.37 puis 3.2.38, cependant certaines modifications proposées ne sont pas présentes, en particulier il manque une ligne (en utilisant la même syntaxe que vos corrections) :

Code : Tout sélectionner

if($rparameters['server_proxy_url']) {curl_setopt($c, CURLOPT_PROXY, $rparameters['server_proxy_url']);}
après la ligne 912 de /var/www/html/system.php sans quoi il y a une erreur "Could not resolve host : url", et l'adresse IP (v4 et/ou v6) publique n'est pas affichée, due à l'utilisation de file_get_contents, ce qui ne passe par le serveur proxy, alors que nous proposions d'utiliser curl et le nouveau paramètre server_proxy_url, par exemple dans une fonction dédiée PublicIP.
Fichiers joints
erreur_could_not_resolve_host.jpg
erreur_could_not_resolve_host.jpg (55.82 Kio) Vu 740 fois
Avatar du membre
Flox
Administrateur du site
Messages : 9049
Enregistré le : jeu. 21 juin 2012 19:00

Bonjour,

une correction sera disponible dans la prochaine version 3.2.39.

Cdt
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.58 | MariaDB: 11.3.2 | PHP: 8.3.6 | https://doc.gestsup.fr/
tom.ricci
Gsup LEVEL 0
Messages : 7
Enregistré le : dim. 2 juil. 2023 17:25

Bonjour,

Merci par avance.

Bonne fin de journée !
Répondre