Download ZIP archive keeps file in /tmp open and results in out of disk space

Issue #448 resolved
grossh created an issue

We are downloading files via the download archive function on the summary page. After some downloads we get out of disk space on /tmp. We found that the zip archive prepared in /tmp is deleted but obviously not released after the download has been completed. This can be seen by using 'lsof | grep /tmp'. If the server is freshly startet, no entry can be seen. After each download one open file in the size of the zip archive marked as deleted is added to the list. As far as we can see, the entries only vanish performing a server restart.

Comments (5)

  1. Marcin Kuzminski repo owner

    Hmm that is something magical, i was able to reproduce this issue but only when you download with browser, using wget doesn't leave anything unreleased. I tried few different variants of downloading archive, and always have this issue, for now i'm clueless how to resolve this

  2. Marcin Kuzminski repo owner
    • changed status to open

    Ok, so i think i narrowed down the problem to be caused by waitress http server. Are you using it ? Using paste#http or gunicorn_paster i don't have this issue. If you use waitress gunicorn is a good alternative for it.

  3. grossh reporter
    • changed component to UNIX
    • changed version to 1.3.2

    In our production.ini I find the following line:

    use = egg:Paste#http

    So I would asume we use paste#http.

    We have an apache http server upfront configured for https using reverse proxy

  4. Marcin Kuzminski repo owner

    I think this issue is now solved in beta branch, due to fix for #455, on unix the same issue results in a delete and no-release, but on windows it throws an exception. Now both should be fixed (and according to my tests there are)

    Would be nice if you can confirm this.

  5. grossh reporter

    Hi, I can confirm that the current beta has solved this problem.

    So we are looking forward to the release containing this.

    Thank you for your support


  6. Log in to comment