[Corrigé 3.2.37] Problème sur images en PJ de Tickets

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.
Répondre
michael.b
Gsup LEVEL 0
Messages : 5
Enregistré le : lun. 22 mai 2023 15:23

Bonjour,

nous nous sommes rendus comptes que les images en PJ de tickets ne sont pas chargées dans le nouvel onglet qui s'ouvre, j'ai arpenté plusieurs fois le forum espérant voir ce bug remonté mais il n'en est rien (ou bien je l'ai raté).

en regardant la page générée j'ai ceci :

Code : Tout sélectionner

<html><head></head><body><img alt="img" src="upload/ticket/2620_4b7117339ab0a8cfe43b47507618f3a0"></body></html>
et en remontant les différentes fonctions, j'arrive à la ligne 41 de core/download.php :

Code : Tout sélectionner

echo '<img alt="img" src="'.$filepath.'" />';


je comprends mieux la génération du lien image qui n'est pas fonctionnel, pour moi, et comme je n'ai pas trouvé de questions à ce sujet, je me doute qu'il doit y avoir un contournement (autre que de dégager la partie qui permet d'afficher les images et télécharger le reste et de simplement tout télécharger) ou un soucis au niveau de ma configuration ^^

pour la configuration actuelle :
Client : différents navigateurs dont Chrome à jour

Serveur :
OS : Windows NT SRV-DOC 10.0 build 14393 (Windows Server 2016) AMD64 (les environnements Windows ne sont pas recommandés pour les serveurs de production)
IP publique : *.*.*.*
Microsoft-iis : 10.0 (en HTTP/1.1 sur *****)
MySQL : 8.0.33 (Il est recommandé de migrer sur MariaDB, base : ****** 46.8MB)
PHP : 8.2.5 (C:\PHP\php.ini)
GestSup : 3.2.35 (2489 tickets / 0 équipements)

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

Bonjour,

pouvez-vous reproduire le problème sur la version de démonstation.

cdt
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.59 | MariaDB: 11.5.2 | PHP: 8.3.12 | https://doc.gestsup.fr/
michael.b
Gsup LEVEL 0
Messages : 5
Enregistré le : lun. 22 mai 2023 15:23

je viens de faire le test sur la version de démo avec un jpg quelconque, l'image s'affiche correctement, bien que le lien dans les source pointe directement le fichier sans extension comme sur mon post initial

d'autres tests que je pourrais effectuer ?

merci :)
Avatar du membre
Flox
Administrateur du site
Messages : 9456
Enregistré le : jeu. 21 juin 2012 19:00

Effectivement c'est normal pour des raisons de sécurité les fichiers uploadés sont renommés.

Pour avancer dans le débogage pourrez-vous transmettre une impression écran de votre page système.
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.59 | MariaDB: 11.5.2 | PHP: 8.3.12 | https://doc.gestsup.fr/
michael.b
Gsup LEVEL 0
Messages : 5
Enregistré le : lun. 22 mai 2023 15:23

Les voici (amputés de quelques informations)
gestsup-sys1.png
gestsup-sys1.png (145.26 Kio) Vu 1490 fois
gestsup-sys2.png
gestsup-sys2.png (153.62 Kio) Vu 1490 fois
cdlt
Avatar du membre
Flox
Administrateur du site
Messages : 9456
Enregistré le : jeu. 21 juin 2012 19:00

Votre serveur web n'est pas supporté pouvez-vous basculer sur Apache et recommencer
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.59 | MariaDB: 11.5.2 | PHP: 8.3.12 | https://doc.gestsup.fr/
michael.b
Gsup LEVEL 0
Messages : 5
Enregistré le : lun. 22 mai 2023 15:23

je n'ai pas la possibilité de faire le test sur apache sur ce serveur, je viens pas contre de tester en modifiant le code de core/download.php en remplaçant la ligne 41 par :

Code : Tout sélectionner

header('Content-Type: image/jpeg');
echo file_get_contents($filepath);
et l'affichage de l'image est alors fonctionnel

je ne sais pas si cette intégration pourrait être fonctionnelle sous apache vu que je ne peux pas la tester, mais le cas échéant, pensez-vous que ça puisse être intégré ultérieurement ?

Cdlt
Avatar du membre
Flox
Administrateur du site
Messages : 9456
Enregistré le : jeu. 21 juin 2012 19:00

La modification sera disponible sur la prochaine version du logiciel.
GestSup: 3.2.47 | Debian: 12 | Apache: 2.4.59 | MariaDB: 11.5.2 | PHP: 8.3.12 | https://doc.gestsup.fr/
michael.b
Gsup LEVEL 0
Messages : 5
Enregistré le : lun. 22 mai 2023 15:23

Bonjour,

merci pour votre retour et l'intégration à la nouvelle mouture.

Cdlt
Répondre