Download files through FTP (php)

bgallz

I currently have a script that downloads a file like so:

	header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
	header("Content-Description: File Transfer");
	header("Pragma: public");
	header("Expires: 0");
	header('Content-Type: '.$mime.'');  

	header('Content-Disposition: attachment; filename="'.basename($source).'"'); 
	header('Content-Length: '.filesize(ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id'])).'');
	header('Content-Transfer-Encoding: binary');
	@readfile($source);

This works fine for downloading files. However...

I contacted my host and told them that the downloads were failing to finish larger than normal downloads. Pretty much anything over like 200-250 MB. They said the download max limit was set to no limit so it can't be that.

They said to try and use ftp connection to download the files... Well...

How exactly do you do that?

I have seen php scripts that connect with ftp login info but none that DOWNLOAD files to ones computer through ftp connection...

Thanks!!!

dagon

check out the ftp section in the manual

bgallz

Okay, I tried using ftp_get to download it. Here's what I have:

    header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
	header("Content-Description: File Transfer");
	header("Pragma: public");
	header("Expires: 0");
    header('Content-Type: '.$mime.'');  

	header('Content-Disposition: attachment; filename="'.basename($source).'"'); 
	header('Content-Length: '.filesize(ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id'])).'');
	header('Content-Transfer-Encoding: binary');

$ftp = ftp_connect('ftp.***.com') or die('Could not connect to FTP.');
ftp_login($ftp,"*******","*******") or die('Login failed.');
ftp_get($ftp, basename($source), ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id']), FTP_BINARY);
ftp_close($ftp);

It downloads a file named that (.txt) with this error inside:

<br />
<b>Warning</b>:  filesize() [<a href='function.filesize'>function.filesize</a>]: stat failed for /home/bgallz/public_html/downloads/patches/BF2_Patch_1.41.exe in <b>/home/bgallz/public_html/download_file.php</b> on line <b>33</b><br />
<br />
<b>Warning</b>:  Cannot modify header information - headers already sent by (output started at /home/bgallz/public_html/download_file.php:33) in <b>/home/bgallz/public_html/download_file.php</b> on line <b>33</b><br />
<br />
<b>Warning</b>:  Cannot modify header information - headers already sent by (output started at /home/bgallz/public_html/download_file.php:33) in <b>/home/bgallz/public_html/download_file.php</b> on line <b>34</b><br />
<br />
<b>Warning</b>:  ftp_get() [<a href='function.ftp-get'>function.ftp-get</a>]: Can't open /home/bgallz/public_html/downloads/patches/BF2_Patch_1.41.exe: No such file or directory in <b>/home/bgallz/public_html/download_file.php</b> on line <b>38</b><br />

🙁

Weedpacket

Warning: filesize() [<a href='function.filesize'>function.filesize</a>]: stat failed for /home/bgallz/public_html/downloads/patches/BF2_Patch_1.41.exe in /home/bgallz/public_html/download_file.php on line 33

The rest are consequences of the first one (except the last one, which we'll come to)
I'm going to guess that line 33 is this one:

	header('Content-Length: '.filesize(ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id'])).'');

What does getFile() return? You're using it like a filename.

Also, it looks like you're trying to download the file twice, the first time just to get its size.

Suggestion: download the file, save it in a temp file, get the size from that, and then output all the HTTP headers followed by the file content itself.

bgallz

The function "getFile()" returns the filename of the file. Something like file.zip.

What function saves it to a temporary file?

Once I save it to a temporary file do I just use the filesize() function and HTTP headers all on that temporary file?

Thanks for the help!

Weedpacket

bgallz wrote:
The function "getFile()" returns the filename of the file

So the function should really be called "getFilename()" 🙂

What function saves it to a temporary file?

[man]tmpfile[/man]

Once I save it to a temporary file do I just use the filesize() function and HTTP headers all on that temporary file?

yes.

bgallz

I'm trying to figure out how to make this work how you're saying...

So I create the temporary file...

$temp = tmpfile();
fwrite($temp, file_get_contents($file));

Somewhere at the end...

fclose($temp);

How do I write to the file if I am using this file to download...

How does this connect with ftp connection?

Sorry, little confused. 😛

Weedpacket

Download the file using ftp.
Save its contents in a temporary file.
Read the file size of the temporary file and output it in the headers.
Output the contents of the temporary file.

bgallz

Weedpacket;10904276 wrote:

Download the file using ftp.

Save its contents in a temporary file.

Read the file size of the temporary file and output it in the headers.

Output the contents of the temporary file.

So to get this straight...lol something like this:

	$temp = tmpfile();
	$ftp = ftp_connect('ftp.bfgamerz.com') or die('Could not connect to FTP.');
	ftp_login($ftp,"*****","*****") or die('Login failed.');
	ftp_get($ftp, $temp, ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id']), FTP_BINARY);
	ftp_close($ftp);

header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
header("Content-Description: File Transfer");
header("Pragma: public");
header("Expires: 0");
header('Content-Type: '.$mime.'');  
header('Content-Disposition: attachment; filename="'.basename($source).'"'); 
header('Content-Length: '.filesize(ROOT . DIR_DOWNLOADS . $cat .'/' . $Downloads->getFile($_GET['id'])).'');
header('Content-Transfer-Encoding: binary');

@readfile($temp);

I also tried this as well, no go:

	$ftp = ftp_connect('ftp.bfgamerz.com') or die('Could not connect to FTP.');
	$login = ftp_login($ftp,"*****","*****") or die('Login failed.');
	$temp = tmpfile();
	ftp_chdir($ftp,ROOT . 'downloads/'.$cat.'/');

if($login){
	ftp_fget($ftp, $temp, $Downloads->getFile($_GET['id']), FTP_BINARY);
} else {
	die('Login failed.');
}
ftp_close($ftp);
@readfile($temp);
fclose($temp);

Which returns the following:

Warning: ftp_chdir() [function.ftp-chdir]: Can't change directory to /home/bgallz/public_html/downloads/patches/: No such file or directory in /home/bgallz/public_html/download_file.php on line 30

Warning: ftp_fget() [function.ftp-fget]: Can't open bfv_incr_patch_v1.2_to_v1.21.exe: No such file or directory in /home/bgallz/public_html/download_file.php on line 33

? lol I'm trying here. 😛

Weedpacket

Actually that error message makes your original issue a bit clearer. What you're trying to do here isn't going to make any difference: you're still trying to serve the file over http and having problems. Having the server open an ftp connection to itself, download the file to itself, and then serve the downloaded file over http isn't going to change the fact that you're trying to serve a file over http.

I think your host means you provide ftp access to the download area. How you do that is something to talk to them about. If you allow people ftp access then you'd given them an ftp:// URL to the file.