302: excess temp directories left behind

eric****@gmai***** (Google Code) (Is this you? Claim this profile.)
Dec. 24, 2007
I have reviewboard installed using mod_python with apache prefork model. 
The reviewboard server is creating a significant number of temp directories
in /tmp (at least 100 per day).

diffutils.py is explicitly setting the tempfile tempdir variable, but it
never tries to unlink that directory.  Per the Python docs: "The user of
mkdtemp() is responsible for deleting the temporary directory and its
contents when done with it."  I'm guessing some interaction of apache and
mod_python is creating multiple interpreters frequently which causes
reviewboard to set the tempdir variable to a different value each time.

Not sure if it makes sense to have it immediately unlink the dir where it
unlinks the file or to use an atexit hook or something else.
chipx86
#1 chipx86
I thought we used to do this, so I'll have to figure out why we don't now.
  • -Priority-Medium
    +Priority-High
    +Component-DiffParser
chipx86
#2 chipx86
  • +Milestone-Release1.0
david
#3 david
Fixed in SVN.  Thanks!
  • +Fixed