Discussion :

[teddy76db]

Bonsoir à tous,

voilà je suis très embêté mon script fonctionne de téléchargement fonctionne très bien sauf qu'à la fin du téléchargement les fichiers .zip sont corrompus si vous avez l'aimabilité de m'éclairer s'il vous plait merci.

Voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
<?php
$dir = "lol/"; // trailing slash is important
$file = $dir .$_POST['filename'] ;
 
if (file_exists($file)) {
ob_start();
header($_SERVER['SERVER_PROTOCOL'].' 200 OK');
header('Pragma: public');
header("Expires: 0");
header("Accept-Ranges: bytes");
header("Connection: keep-alive");
header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
header("Cache-Control: public");
header("Content-Transfer-Encoding: binary");
header('Content-Description: File Transfer');
header("Content-type: application/zip");
header('Content-Disposition: attachment; filename='.basename($file));
header('Expires: 0');
header('Cache-Control: must-revalidate');
header('Content-Length: ' . filesize($file));
ob_clean();
readfile($file);
 
} else {
echo "Le fichier $file n'existe pas.";
}
exit;
 
?>

J'utilise la méthode POST filename afin de récupérer le nom du fichier téléchargé sur mon site, cordialement.

[grunk]

Que contient le zip quand tu l'ouvre avec un éditeur de text classique ?
Peut être y'a t'il une erreur php écrite en claire , ce qui expliquerais la corruption du fichier.

L'utilisation des fonction de bufferisation ob_* n'est pas nécessaire ici.

[Celira]

L'utilisation des fonction de bufferisation ob_* n'est pas nécessaire ici.

Je dirais même plus : l'utilisation des fonctions ob_* est peut-être la cause du problème : tu utilises ob_clean qui vide le tampon sans l'envoyer au navigateur et tu n'as aucune fonction d'envoi du tampon (ob_​flush ou ob_​end_​flush)

[teddy76db]

Merci pour ta réponse cette fois-ci avec ce code le téléchargement se fait mais avant la fin cela télécharge le zip à 578ko sur 600ko, avec pour erreur : erreur réseau :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
<?php
$dir = "lol/"; // trailing slash is important
$file = $dir .$_POST['filename'] ;
 
if (file_exists($file)) {
ob_start();
header("HTTP/1.1 200 OK");
header('Pragma: public');
header("Expires: 0");
header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
header("Cache-Control: public");
header("Content-Transfer-Encoding: binary");
header("Content-type: application/zip");
header('Content-Disposition: attachment; filename='.basename($file));
header('Cache-Control: must-revalidate');
header('Content-Length: ' . filesize($file));
ob_end_flush();
readfile($file);
 
} else {
echo "Le fichier $file n'existe pas.";
}
exit;
 
?>

[grunk]

A quoi te servent tes fonctions ob_* ?

[teddy76db]

A quoi te servent tes fonctions ob_* ?

Purger le cache, j'ai déjà essayé sans, ça ne fonctionne pas pour autant :/

[Celira]

Quel cache ? les fonctions ob_* gèrent un tampon, autrement dit on envoie le contenu du tampon d'un seul coup au lieu d'envoyer ligne à ligne.

Sinon, quelques détails à propos des headers :
Personnellement, j'aurais utilisé header("Content-type: application/octet-stream"); à la place de header("Content-type: application/zip");.
Il manque les quotes autour du nom du fichier : header('Content-Disposition: attachment; filename="'.basename($file).'"');.