[Rt-commit] [svn] r1072 - rt/branches/rt-3.1/lib/RT

[jesse at pallas.eruditorum.org]

Author: jesse
Date: Mon Jun 14 20:08:15 2004
New Revision: 1072

Modified:
   rt/branches/rt-3.1/lib/RT/EmailParser.pm
Log:
RT-Ticket: 5712

It turns out that MIME::Parser has a lame, non-obvious default for tempfile reuse.
Perhaps code-hits will fix our problem.




Modified: rt/branches/rt-3.1/lib/RT/EmailParser.pm
==============================================================================
--- rt/branches/rt-3.1/lib/RT/EmailParser.pm	(original)
+++ rt/branches/rt-3.1/lib/RT/EmailParser.pm	Mon Jun 14 20:08:15 2004
@@ -655,6 +655,15 @@
     # do _not_ store each msg as in-core scalar;
 
     $parser->output_to_core(0);
+
+    # From the MIME::Parser docs:
+    # "Normally, tmpfiles are created when needed during parsing, and destroyed automatically when they go out of scope"
+    # Turns out that the default is to recycle tempfiles
+    # Temp files should never be recycled, especially when running under perl taint checking
+    
+    $parser->tmp_recycling(0);
+    
+
 }
 
 # }}}