maryjane/gpodder
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Wiki Activity

Support downloads without content-length header (bug 535)

Browse source 
Some servers, like the one serving "Bloomberg" episodes
do not send a "Content-length" header in their HTTP
response, which made gPodder fail to download files.

This patch fixes the issue and also re-calculates the
file size after the download has completed, so the real
file size appears after the file has been downloaded.
This commit is contained in:
Thomas Perl 2009-09-01 14:37:11 +02:00
parent a860f5786e
commit 2e20b27fdc
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/gpodder/download.py
View file

@ -249,6 +249,7 @@ class DownloadURLOpener(urllib.FancyURLopener):
# gPodder TODO: we can get the real url via fp.geturl() here
# (if anybody wants to fix filenames in the future)
# Maybe also utilize the name in the "Content-disposition" header
result = filename, headers
bs = 1024*8
@ -259,7 +260,7 @@ class DownloadURLOpener(urllib.FancyURLopener):
if "content-length" in headers:
size = int(headers["Content-Length"]) + current_size
reporthook(blocknum, bs, size)
while read < size:
while read < size or size == -1:
if size == -1:
block = fp.read(bs)
else:
@ -648,6 +649,9 @@ class DownloadTask(object):
if self.status == DownloadTask.DOWNLOADING:
# Everything went well - we're done
self.status = DownloadTask.DONE
if self.total_size <= 0:
self.total_size = util.calculate_size(self.filename)
log('Total size updated to %d', self.total_size, sender=self)
self.progress = 1.0
return True