uploading files to file system/zipping/downloading problems

[OriginalBrownster]

I am currently uploading a file from a users computer to the file
system on my server using python, just reading the file and writing the
binaries.

total_data=' '
while True:
data = upload_file.fil e.read(8192)
if not data:
break
total_data += data
f = open(target_fil e_name, 'wb')
f.write(total_d ata)
f.close

However when i download the file from the server it is not intact and
it cannot be opened. It is happening with every type of file.

It seemed to be working before and now it is..maybe I goofed up and
deleted something.
However I can't seem to find it.

any ideas??

[Pontus Ekberg]

On Sun, 20 Aug 2006 13:49:22 -0700, OriginalBrownst er wrote:

I am currently uploading a file from a users computer to the file
system on my server using python, just reading the file and writing the
binaries.

total_data=' '
while True:
data = upload_file.fil e.read(8192)
if not data:
break
total_data += data
f = open(target_fil e_name, 'wb')
f.write(total_d ata)
f.close

However when i download the file from the server it is not intact and
it cannot be opened. It is happening with every type of file.

It seemed to be working before and now it is..maybe I goofed up and
deleted something.
However I can't seem to find it.

any ideas??

You are re-creating, and overwriting, your file every time you go into
that while loop.

[Jordan]

Assuming your upload_file.fil e.read() function works, the overwriting
should be the only issue. Just want to point out that total_data +=
data is not advisable for this example (speed/efficiency issue,
although i'm sure it could probably be even faster than what I replace
it with if u decided to use arrays, but it's probably not worth it XD),
and I think open() was replaced by file(), although they're probably
interchangable.

new code
-----------------
total_data = ' '
temp_data = []
while True:
data = upload_file.fil e.read(8192)
if not data:
break
temp_data.appen d(data)
total_data = ' '.join(temp_dat a)
somefile = file(target_fil e_name, 'wb')
somefile.write( total_data)
f.close()
-------------------

OriginalBrownst er wrote:

I am currently uploading a file from a users computer to the file
system on my server using python, just reading the file and writing the
binaries.

total_data=' '
while True:
data = upload_file.fil e.read(8192)
if not data:
break
total_data += data
f = open(target_fil e_name, 'wb')
f.write(total_d ata)
f.close

However when i download the file from the server it is not intact and
it cannot be opened. It is happening with every type of file.

It seemed to be working before and now it is..maybe I goofed up and
deleted something.
However I can't seem to find it.

any ideas??

[Larry Bates]

OriginalBrownst er wrote:

I am currently uploading a file from a users computer to the file
system on my server using python, just reading the file and writing the
binaries.

total_data=' '
while True:
data = upload_file.fil e.read(8192)
if not data:
break
total_data += data
f = open(target_fil e_name, 'wb')
f.write(total_d ata)
f.close

However when i download the file from the server it is not intact and
it cannot be opened. It is happening with every type of file.

It seemed to be working before and now it is..maybe I goofed up and
deleted something.
However I can't seem to find it.

any ideas??

Two problems:

First, If you start total_data with a single space (as it shows in your
posted code). Then the output file has a space prepended to the
file's contents and that is NOT what you probably wanted.

Secondly, You are overwriting the files contents every time through
the loop. Your open, would need to be outside the loop (above the
while) and your close should be outside the loop also. Something more
like:

total_data=''
f = open(target_fil e_name, 'wb')

while True:
data = upload_file.fil e.read(8192)
if not data:
break
total_data += data
f.write(total_d ata)

f.close

You should take a look at shutil module. It is copyfile method
that makes your code must simpler (and faster I'm sure).

do

import shutl
help(shutil.cop yfile)

import shutil
shutil.copyfile (uload_file_nam e, target_file_nam e)
-Larry Bates