In the past I ran into the problem that session handling and the
method serve_download() doesn't work well
together. The download of files bigger 100 MB results in an empty file
downloaded by the browser.
I got no error, neither in the cherrypy-log nor on the browser side.
Just this empty file saved on the harddisk.

This small test script should whow this behaviour: (note: you need a
big file to serve)

import os
import cherrypy
from cherrypy.lib.static import serve_download

THIS_DIR = os.path.abspath(os.path.dirname(__file__))

class HelloWorld(object):
    _cp_config = {"tools.sessions.on": True}
    #_cp_config = {} # uncomment this and download will work fine
    def index(self):
        try:
            cherrypy.session["hello"] = "hello"
        except:
            pass
        return serve_download(os.path.join(THIS_DIR, "bigfile.bin"))
    index.exposed = True
cherrypy.config.update({'server.socket_host': '0.0.0.0',
                        'server.socket_port': 8080,
                       })
cherrypy.quickstart(HelloWorld())

Is this a known issue or a limitation we have to live with?

My workaround makes a HTTPRedirect to a download method which is not
part of an object with session handling activated. This seems to work
in general but some clients told me they still download files with a
size of zero bytes sometimes. After some retries the download will
work right but it seems to be some kind of unstable.
Do you have any experience with this unstable behaviours, too?

Best regards,
Zap

--dirk

Perhaps you're interested to assume the procedure in the cherrypy
sources.
Fact is, I could reproduce the bug on different pc's running
WindowXP
and this workaround helps so far.

import socket
from cherrypy.wsgiserver import CP_fileobject

CP_fileobject_sendall = CP_fileobject.sendall
def _sendall(self, data):
    """ Monkeypatch for cherrypy ticket #909 """
    try:
        CP_fileobject_sendall(self, data)
    except socket.error, e:
        if e.args[0] == 10055:
            # 10055 -> "No buffer space available"
            # try to send the data in smaller parts
            block = 1024 ** 2 # 1 MB
            for pos in xrange(0, len(data), block):
                CP_fileobject_sendall(self, data[pos:pos+block])
        else:
            raise
CP_fileobject.sendall = _sendall

Zap

On 7 Mai, 06:03, Zap <z...@lancraft.net> wrote: