[RESOLU]Content-types et upload d'image - Librairie GD

Content-types et upload d'image - Librairie GD [RESOLU] - PHP - Programmation

Marsh Posté le 15-01-2009 à 11:45:56    

Bonjour à tous,
Décidément c' est pas ma semaine. :pfff:  
 
Voilà j'ai un p'ti soucis. :(  
J'ai un script qui doit enregistrer des infos sur une personne dans une BDD, et dans ces infos il y a aussi une photo dans le formulaire.
(jusque là c'est pas extraordinaire)
 
Alors j'ai créé un petit script, pour l'upload de la photo, qui retaille la photo 2x.
Ce script stocke 1 miniature dans un dossier photo/mini.
Et il stocke l'autre dans un dossier photo/normal
 
Les images sont bien retaillées, et elles sont bien envoyées dans leur dossier respectif.
Sauf que :  
 
J'ai l'impression qu'à la fin de ce script de redimensionnement, le content-type reste bloqué sur image/jpeg.  :pt1cable:  
 
La suite du code ne s'execute pas, la page se fige sur une page blanche avec comme en-tête mapage.php (image JPEG) et affiche l'url de la page que j'ai demandée. :??:  
 
Structure du code :

Code :
  1. /* Content-type */
  2. header('Content-type: image/jpeg');
  4.    redimensionnement();
  5.    déplacement();
  7. /* Content-type */
  8. header('Content-type: text/html');
  10. /* Suite du code */
  11.    affichage_html_divers();
  12.    enregistrement_db();


 
qques lignes de mon phpinfo(); :
-PHP Version 4.3.9
-GD Support enabled  
-GD Version bundled (2.0.28 compatible)
 
Ce script de redimensionnement, je l'ai déjà utilisé pour un autre site et il fonctionne correctement. :heink:  
Si quelqu'un a déjà eu ce problème... ;)  
 
Ju.


Message édité par jocastt le 15-01-2009 à 14:32:39
Reply

Marsh Posté le 15-01-2009 à 11:45:56   

Reply

Marsh Posté le 15-01-2009 à 12:04:18    

Il sert à quoi ton premier Content-Type ? Parce que là il ne sert strictement à rien, si t'affiches de l'HTML après [:pingouino]

Message cité 1 fois
Reply

Marsh Posté le 15-01-2009 à 12:07:07    

FlorentG a écrit :

Il sert à quoi ton premier Content-Type ? Parce que là il ne sert strictement à rien, si t'affiches de l'HTML après [:pingouino]


 
Les fonctions de créations d'images via librairie GD ne sont disponibles qu'en content-type image/xxx non ?

Reply

Marsh Posté le 15-01-2009 à 12:08:39    

Nan mais le content-type est destiné au navigateur. Que ce soit PHP ou GD, ils s'en foutent. Ils ne regardent même pas ce header. Ca n'intervient pas du tout dans l'exécution du script, c'est uniquement le navigateur qui l'utilise pour déterminer le type de contenu.

Message cité 1 fois
Reply

Marsh Posté le 15-01-2009 à 12:12:21    

FlorentG a écrit :

Nan mais le content-type est destiné au navigateur. Que ce soit PHP ou GD, ils s'en foutent. Ils ne regardent même pas ce header. Ca n'intervient pas du tout dans l'exécution du script, c'est uniquement le navigateur qui l'utilise pour déterminer le type de contenu.


Comment tu peux créer une image jpeg via php sans spécifier une en-tête pendant la création de l'image en question. :??:  
 :non: Je ne suis pas sur que ca soit possible...

Message cité 1 fois
Reply

Marsh Posté le 15-01-2009 à 12:16:25    

jocastt a écrit :

Comment tu peux créer une image jpeg via php sans spécifier une en-tête pendant la création de l'image en question. :??:


Tu confonds tout en fait. Les fonctions GD s'en foutent de l'entête. Elles ne savent même pas qu'il existe.

 

L'entête est uniquement utilisé lorsque tu affiches une image, genre :

Code :
  1. header('Content-Type: image/jpeg');
  2. readfile('monimage.jpg');
 

Donc quand tu réceptionnes les données du formulaire, tu fais l'insertion en base, puis le redimensionnement + sauvegarde de l'image, et c'est tout. Mais le redimensionnement ou la sauvegarde s'opèrent sur un fichier, donc il n'y a pas de notion d'HTTP, donc pas d'entête Content-Type

Message cité 1 fois
Message édité par FlorentG le 15-01-2009 à 12:17:08
Reply

Marsh Posté le 15-01-2009 à 12:26:37    

FlorentG a écrit :


Tu confonds tout en fait. Les fonctions GD s'en foutent de l'entête. Elles ne savent même pas qu'il existe.
 
L'entête est uniquement utilisé lorsque tu affiches une image, genre :


Tu pourras donc aisément m'expliquer pourquoi ce code fonctionne :  

Code :
  1. header('Content-type: image/jpeg');
  2. list($width, $height) = getimagesize($img);
  3. $ratio = $width / $height;


Et pourquoi celui là ne fonctionne pas : (Division by zero in xxx.php on line 2)

Code :
  1. list($width, $height) = getimagesize($img);
  2. $ratio = $width / $height;


 :sarcastic:  

Reply

Marsh Posté le 15-01-2009 à 12:28:22    

Euh chez moi ça marche très bien, il n'y a vraiment pas besoin de header. Le problème est ailleurs.
 
Le header n'est pas du tout lu par PHP. Si t'en veux la preuve, je te sors le code source...

Reply

Marsh Posté le 15-01-2009 à 12:29:51    

En fait c'est simple d'ailleurs, le premier ne fonctionne pas non-plus. Mais comme c'est un header différent de text/html, le navigateur n'affiche pas le texte, croyant que c'est une image.

 

Sniff le traffic HTTP, tu verra que le message d'erreur y est aussi


Message édité par FlorentG le 15-01-2009 à 12:30:06
Reply

Marsh Posté le 15-01-2009 à 12:35:10    

Alors je continue de fouiller sur le net en même temps,
voilà une petite page que j'ai trouvée..
 
En haut de chacun de leur script, j'ai l'impression de voir un header.   :sweat:  
 
Par contre, je veux bien voir le code qui génère des images jpeg, sans que l'en-tête jpeg soit appelé. (si ca te dérange pas trop...)

Reply

Marsh Posté le 15-01-2009 à 12:35:10   

Reply

Marsh Posté le 15-01-2009 à 12:38:11    

Si eux mettent un entête en haut, c'est parce qu'ils affichent tout de suite après l'image. Sachant qu'il faut préciser le Content-Type pour que le navigateur sache comme l'afficher.
 
Mais nul besoin de ces entêtes pour redimensionner/sauvegarder une image :) Ton code :

Code :
  1. list($width, $height) = getimagesize($img);
  2. $ratio = $width / $height;


Il faut juste s'assurer que $img contient le chemin vers l'image

Reply

Marsh Posté le 15-01-2009 à 12:42:15    

Purée là je suis perdu, et en plus j'ai faim quoi...
$img contient bien le lien vers l'image en question. Bizarre cette division par 0. :??:  
 
Il faut m'aider ! :p :D
 
Bon, j'ai horreur de faire ca... Mais peut-être l'erreur sautera aux yeux de quelqu'un.

Code :
  1. if (is_uploaded_file($_FILES['photo']['tmp_name'])){
  2. $error = '';
  4. $ftmp  = $_FILES['photo']['tmp_name'];
  5. $fname = (substr(md5(uniqid(rand(), true)),0,6)).(str_replace(' ','_', $_FILES['photo']['name']));
  6. $fname = strtr($fname,'àáâãäçèéêëìíîïñòóôõöùúûüýÿÀÁÂÃÄÇÈÉÊËÌÍÎÏÑÒÓÔÕÖÙÚÛÜÝ','aaaaaceeeeiiiinooooouuuuyyAAAAACEEEEIIIINOOOOOUUUUY');
  7. $fname = strtolower($fname);
  9. //extensions autorisées  
  10. $extension_autorises= array('.jpg','.jpeg');
  11. $extension= strrchr($fname,'.');
  13. //lien des photos
  14. $lien    = 'photo/veto/normal/'.$fname;
  16. //dossier miniature
  17. $miniature = 'photo/veto/mini/'.$fname;
  19. /*Definition de la taille max des minatures. */
  20. $width     = 100;
  21. $height    = 100;
  23. $width_n   = 600;
  24. $height_n  = 600;
  26. //verification de l'extension
  27. if (!in_array($extension,$extension_autorises)){
  28.  $error .= 'Le fichier n\'est pas une image';
  29. }
  31. if (!move_uploaded_file($ftmp,$lien)){
  32.  $error .= 'Impossible de copier le fichier';
  33. }
  35.         /* Content type */
  36.         header('Content-type: image/jpeg');
  39.         /*creation des miniatures. */
  40. list($width_orig, $height_orig) = getimagesize($lien);
  42. // On stoppe le script si la taille de la photo est trop grande
  43. if ($width_orig > 1000 || $height_orig > 1000){
  44.  exit('L\'image1 est trop grande <br /><a href="javascript:history.back();">retour</a><br />'.$rappel.' <hr />');
  45. }
  46. $ratio_orig   = $width_orig/$height_orig;
  48. // Ratio miniature
  49. if ($width_max/$height_max > $ratio_orig){
  50.  $width    = $height*$ratio_orig;
  51. }else{
  52.  $height   = $width/$ratio_orig;
  53. }
  54. // Ratio normal
  55. if ($width_orig > $width_n || $height_orig > $height_n){
  56.  if ($width_n/$height_n > $ratio_orig){
  57.   $width_n  = $height_n*$ratio_orig;
  58.  }else{
  59.   $height_n = $width_n/$ratio_orig;
  60.  }
  61. }else{
  62.  $width_n = $width_orig;
  63.  $height_n = $height_orig;
  64. }
  66. // Config miniature
  67. $image_p   = imagecreatetruecolor($width,$height);
  68. $image    = imagecreatefromjpeg($lien);
  69. imagecopyresampled($image_p,$image,0,0,0,0,$width, $height, $width_orig, $height_orig);
  71. // Config normale
  72. $image_p_n    = imagecreatetruecolor($width_n,$height_n);
  73. $image_n      = imagecreatefromjpeg($lien);
  74. imagecopyresampled($image_p_n,$image_n,0,0,0,0,$width_n, $height_n, $width_orig, $height_orig);
  76. //movance  de la miniature crée au dessus dans le repertoire miniature  
  77. imagejpeg($image_p,$miniature,90);
  79. //movance  de la miniature crée au dessus dans le repertoire normal  
  80. imagejpeg($image_p_n,$lien,90);
  82. imagejpeg($image_p);
  84. unset($miniature);
  85. unset($image_p);
  87. unset($image_p_n);
  88. unset($lien);
  90. unset($image);
  91. unset($width);
  92. unset($height);
  93. unset($image_n);
  94. unset($width_n);
  95. unset($height_n);
  96. unset($ratio_orig);
  98. }else{
  99. $fname='';
  100. }


 
 :sweat: désolé.


Message édité par jocastt le 15-01-2009 à 13:16:31
Reply

Marsh Posté le 15-01-2009 à 14:01:02    

Ton code là :

Code :
  1. list($width_orig, $height_orig) = getimagesize($lien);


$lien doit être le fichier image, là c'est totalement autre chose. Il faut remplace par :
 

Code :
  1. list($width_orig, $height_orig) = getimagesize($_FILES['photo']['tmp_name']);


 
Et pareil après dans l'appel à imagecreatefromjpeg.
 
Aussi, la validation du type d'image (basée sur l'extension) n'est pas bonne et dangereuse. N'importe qui peut renommer un fichier en .jpg. Pareil si l'image est invalide. Il faut plutôt vérifier le type retourné par getimagesize

Message cité 1 fois
Reply

Marsh Posté le 15-01-2009 à 14:23:11    

FlorentG a écrit :

Ton code là :

Code :
  1. list($width_orig, $height_orig) = getimagesize($lien);


$lien doit être le fichier image, là c'est totalement autre chose. Il faut remplace par :

Code :
  1. list($width_orig, $height_orig) = getimagesize($_FILES['photo']['tmp_name']);




 
un echo de <img src=$lien /> m'affiche bien l'image uploadée.
(car je bouge le fichier temp vers le dossier de mon choix grace a move_uploaded_file)
Donc $lien contient bien le lien relatif de mon image. :(  
 
Merci pour ton aide en tout cas. Je commence à perdre espoir. :pfff:  
Question sécurité, tu as tout à fait raison et je me pencherai sur la question dès que ce contre temps sera reglé.
 
Si quelqu'un voit ce qui cloche..., qu'il n'hésite pas :s

Reply

Marsh Posté le 15-01-2009 à 14:24:12    

Peut-être avec un chemin absolu plutôt que relatif ? Ce n'est pas sûr que getimagesize gère tout ce qui est include path et tout.

Message cité 1 fois
Reply

Marsh Posté le 15-01-2009 à 14:29:27    

FlorentG a écrit :

Peut-être avec un chemin absolu plutôt que relatif ? Ce n'est pas sûr que getimagesize gère tout ce qui est include path et tout.


Laisse tomber je viens de voir mon erreur. Ca devient grave... Regarde ligne 49. 2 variables qui ne sont pas déclarées . . . :o  
Comment perdre 2 heures pour une connerie pareille :lol:  
 
Ca me rend dingue tout ca. :cry:  
 
En tout cas merci pour ton aide.
 
PS : Bizarrement ca fonctionne beaucoup mieux comme ca. :hello:  :hello:


Message édité par jocastt le 15-01-2009 à 14:31:39
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed