]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/unified_temporary_file__47__directory_handling.mdwn
Merge commit 'upstream/master' into prv/po
[git.ikiwiki.info.git] / doc / todo / unified_temporary_file__47__directory_handling.mdwn
index 99bacf0971aead691098b9fd0cf39c7d59350e20..ca63fbeea730886f25488c8837a81fc6ea254b1b 100644 (file)
@@ -3,3 +3,17 @@ typical place for security bugs.  Would it be worthwhile to have library functio
 and directories?  If nothing else, it would serve as documentation of the "official way".
 
 Off to cut and paste :-) --[[DavidBremner]]
+
+> Hmm, I see only three users of temp files in all ikiwiki:
+> * hnb uses `File::Temp::mkstemp` to create two temp file handles.
+> * teximg uses `File::Temp::tempdir` to create a temporary directory.
+> * attachment retrieves a temp file from `CGI::tmpFileName`.
+> These are three quite different uses of temp files, not subject to
+> unification. Using `File::Temp` (and avoiding the posibly insecure
+> `mktemp`, `tmpname`, and `tempnam` functions) is probably as unified as
+> can be managed. --[[Joey]]
+
+>> OK, fair enough. Somehow the code in teximg made me think it was
+>> all a bit complicated. But after I played with it a bit more (and used File::Temp)
+>> I tend to agree, there is no real problem there to fix.  
+>> Feel free to mark [[done]] [[DavidBremner]]