I've been away from Perl for almost 3 months now. One of my first tasks, now that I'm back, is to create a file upload script. That's not a problem. I've done enough of these that I can code them in my sleep.

Wrong.

What am I missing with the following test case?

#!D:\perl\bin\perl.exe -T
use strict;
use warnings;
use CGI;
use HTML::Entities;

use constant UPLDTMPDIR  => './newDocs/';
use constant ZIPTEMPFILE => 'zip~tempfile.zip';
$CGI::DISABLE_UPLOAD     = 0; # Temporarily reenable uploads
$CGI::POST_MAX           = 1_048_576;

my $cgi = CGI->new;
print $cgi->header,
      $cgi->start_html( -title => "Test" );
get_upload();
print $cgi->h1( "It worked.  Stop whining and get this done!" ),
      $cgi->end_html;

sub get_upload {
    $cgi->cgi_error and error( "Error uploading file: " . $cgi->cgi_er
+ror );

    my $zipFile = $cgi->upload( "zipfile" ) or error( $cgi->p( "No fil
+e uploaded." ) );
    my $format  = $cgi->uploadInfo( $zipFile )->{ 'Content-Type' };

    if ( $format ne 'application/x-zip-compressed' ) {
        error($cgi->p( "Illegal file format." ) );
    }
    open FH, ">", UPLDTMPDIR.ZIPTEMPFILE 
        or die "Can't open ".UPLDTMPDIR.ZIPTEMPFILE." for writing: $!"
+;
    {
        local $/;
        binmode $zipFile;
        print FH <$zipFile>;
    }
    close FH or die "Can't close ".UPLDTMPDIR.ZIPTEMPFILE.": $!";
}
[download]

Everything works perfectly, but I am uploading a zip file that is 31,113 bytes. It's getting saved as 31,217 bytes. When I try to unzip the archive, it's identified as a corrupted. My only guess is that the line endings are being translated, but I thought binmode would take care of that. Heck, I'm embarrassed to be asking this because it's usually the sort of question that I answer :)

Configuration info:

• $CGI::VERSION is 2.74
• perl -v gives This is perl, v5.6.0 built for MSWin32-x86-multi-thread. Build is 620.
• This is being run locally on a Windows 2000 box running IIS 5.0

Cheers,
Ovid

Join the Perlmonks Setiathome Group or just click on the the link and check out our stats.