Since
Item3546 passthrough files are no longer stored in /tmp. This raises the spectre of two problems:
-
configure
will flag an error if the directory where passthrough files are stored does not exist, and /tmp/twiki does not normally exist. This is rather unclean.
-
TWiki::Client::expireDeadSessions
is overloaded to clean up old passthrrough files (which can be left behind when a client aborts a page load) but it only processes the {Sessions}{Dir}
dir. As long as passthrough files were stored there this was ok, but now....
I'm OK with ignoring (1) for now, but passthrough files
must be cleaned up.
--
TWiki:Main/CrawfordCurrie - 01 Feb 2007
In 4.1.0 and earlier I guess we had the same problem. If someone had not put the same directory for passthrough and session files the passthrough files would never be cleaned out.
Maybe we should just have ONE setting for temporary files?
I am currently looking at the passthrough thing in the code to see how they work and if they perhaps should just stay in /tmp
--
TWiki:Main.KennethLavrsen - 01 Feb 2007
At a meeting Monday 01 Feb 2007 in #twiki_security it was decided that
- The $TWiki::cfg{Sessions}{Dir} and $TWiki::cfg{PassthroughDir} are replaced by a new $TWiki::cfg{TempfileDir} which defaults to /tmp/twiki
- The code that create passthrough files must create the TempfileDir like it already happens for sessions in case sessions are disabled.
I have already coded it and will check it in
--
KennethLavrsen - 02 Feb 2007
Done. MAIN and Patch04x01
--
KennethLavrsen - 02 Feb 2007