Problem with my curl download script

jeddiki

I am using curl to down load a zip file and I am
then trying to unzip it, but I don't think that I am
using the functions properly because I get warnings

This is my code:

Expand|Select|Wrap|Line Numbers

 
<?php 
 
if (!extension_loaded('zip')) {

    dl('zip.so');

}
 
$func_path =  "/home/guru54gt5/public_html/im/my_functions.php";

require_once("$func_path");
 
$target_url = "http://www.support-focus.com/test_map.zip";

$userAgent = 'Googlebot/2.1 (http://www.googlebot.com/bot.html)';
 
echo "<br>Starting<br>Target_url: $target_url<br><br>";
 
// make the cURL request to $target_url

$ch = curl_init();

curl_setopt($ch, CURLOPT_USERAGENT, $userAgent);

curl_setopt($ch, CURLOPT_URL,$target_url);

curl_setopt($ch, CURLOPT_FAILONERROR, true);

curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);

curl_setopt($ch, CURLOPT_AUTOREFERER, true);

curl_setopt($ch, CURLOPT_BINARYTRANSFER,true);

curl_setopt($ch, CURLOPT_TIMEOUT, 10);

$page = curl_exec($ch);

if (!$page) {

    echo "<br />cURL error number:" .curl_errno($ch);

    echo "<br />cURL error:" . curl_error($ch);

    exit;

}

curl_close($ch);
 
// Un zip the file 
 
    $zip = new ZipArchive;

    if (! $zip) {

        echo "Could not make ZipArchive object.";

        exit;

    }

    $zip->open("$page");

    $zip->extractTo('./');

    $zip->close();

        echo "Ok!";

?>

The errors I get are in fact warnings:

Warning: ZipArchive::extractTo() [function.ZipArchive-extractTo]: Invalid or unitialized Zip object in /home/guru54gt5/public_html/sys/convert_xml_no1.php on line 40

Warning: ZipArchive::close() [function.ZipArchive-close]: Invalid or unitialized Zip object in /home/guru54gt5/public_html/sys/convert_xml_no1.php on line 41
Ok!

Line 40 and 41 are:

Expand|Select|Wrap|Line Numbers

 
$zip->extractTo('./');

$zip->close();

The script is here:

test script

Is the problem because my $page is not a file ?

I am a bit confused about this so would appreciate some help.

Thanks.

Atli

Hey.

If you want the curl_exec function to actually return the results, you need to set the CURLOPT_RETURNTRANSFER option. If you do not, the results will be printed to the browser. (Which is why you get all those random characters on your page.)

Two questions tho.

Why do you use Curl for this? Couldn't you just as well use something like file_get_contents? (Assuming your configuration allows it to use URLs)
Why do you have the request identify itself as Googlebot?

jeddiki

Hi,

Thanks for your reply.

I am using curl because I want to get used to using it and it is more powerful and faster than file_get_contents ( according to what I read)

The Google_bot - well no reason really, it was in the code that
I took as my example.

I have moved forward with my script
but I am still getting errors.

This is what I have:

Expand|Select|Wrap|Line Numbers

 <?php 
 
if (!extension_loaded('zip')) {

    dl('zip.so');

}
 
$func_path =  "/home/guru54gt5/public_html/im/my_functions.php";

require_once("$func_path");
 
$target_url = "http://www.support-focus.com/test_map.zip";

$userAgent = 'Googlebot/2.1 (http://www.googlebot.com/bot.html)';

$file_zip = "new.zip";

$file_txt = "new.txt";
 
echo "<br>Starting<br>Target_url: $target_url<br><br>";
 
// make the cURL request to $target_url

$ch = curl_init();

$fp = fopen("$file_zip", "w"); 

curl_setopt($ch, CURLOPT_USERAGENT, $userAgent);

curl_setopt($ch, CURLOPT_URL,$target_url);

curl_setopt($ch, CURLOPT_FAILONERROR, true);

curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);

curl_setopt($ch, CURLOPT_AUTOREFERER, true);

curl_setopt($ch, CURLOPT_BINARYTRANSFER,true);

curl_setopt($ch, CURLOPT_TIMEOUT, 10);

curl_setopt($ch, CURLOPT_FILE, $fp);
 
$page = curl_exec($ch);
 
if (!$page) {

    echo "<br />cURL error number:" .curl_errno($ch);

    echo "<br />cURL error:" . curl_error($ch);

    exit;

}

curl_close($ch);
 
echo "<br>Downloaded file: $target_url";

echo "<br>Saved as file: $file_zip";

echo "<br>About to unzip ...";
 
// Un zip the file 
 
$zip = new ZipArchive;

    if (! $zip) {

        echo "Could not make ZipArchive object.";

        exit;

    }

    $zip->open("$file_zip");

    $zip->extractTo("$file_txt");

    $zip->close();
 
echo "<br>Unzipped file to: $file_txt<br><br>";        
 
?>

The result:

Quote:

Starting
Target_url: http://www.support-focus.com/test_map.zip

Downloaded file: http://www.support-focus.com/test_map.zip
Saved as file: new.zip
About to unzip ...
Warning: ZipArchive::extractTo() [function.ZipArchive-extractTo]: Invalid or unitialized Zip object in /home/guru54gt5/public_html/sys/convert_xml_no2.php on line 52

Warning: ZipArchive::close() [function.ZipArchive-close]: Invalid or unitialized Zip object in /home/guru54gt5/public_html/sys/convert_xml_no2.php on line 53

Unzipped file to: new.txt

Even though it says that it has unzipped the file,
it doesn't. :confused:

The new version is on: New Version

Anyone know what I am doing wrong ?

Atli

Ok, it seems the $zip->open() call is failing, but because you don't capture and test the return value of the method, the code just moves on and fails when you try to use the unopened zip object.

You need to capture the error to see what is causing the problem.
Try something like:

Expand|Select|Wrap|Line Numbers

 $zip = new ZipArchive();

if($zip) {

    $result = $zip->open($file_zip);

    if($result === true) {

        $zip->extractTo($file_txt);

        $zip->close();

        echo "Success!";

    }

    else {

        echo "Failed to open file at '$file_zip'. Error code: " . $result;

    }

}

else {

    echo "Could not make ZipArchive object.";

}

$zip = null;

P.S.
You can see the error codes in the comments on the ZipArchive:open manual entry.

Problem with my curl download script

re: Problem with my curl download script

re: Problem with my curl download script

re: Problem with my curl download script

Similar PHP bytes

/bytes/development

/bytes/it

/bytes/about