> On Jun 8, 2007, at 10:09 AM, Tobias Kremer wrote:
> > The manpage of Catalyst::Request::Upload suggests the following:
> > --snip--
> > To specify where Catalyst should put the temporary files, set the
> > 'uploadtmp'
> > option in the Catalyst config. If unset, Catalyst will use the
> > system temp dir.
> > __PACKAGE__->config( uploadtmp => '/path/to/tmpdir' );
> > --snip--
> > Unfortunately this doesn't work for me (all file uploads are placed
> > in the
> > system default location, e.g. /tmp).
> > Now, there's this in Catalyst::Engine:
> > #321: $c->request->{_body}->{tmpdir} = $c->config->{uploadtmp}
> > But HTTP::Body::MultiPart doesn't consider $self->{tmpdir} in its
> > call to
> > File::Temp. Looks like tmpdir isn't used at all and really should
> > be added
> > as an object method to avoid poking in the HTTP::Body object.
> > Have I misunderstood the purpose of uploadtmp? If not, I'd be happy to
> > provide patches for HTTP::Body::MultiPart and Catalyst::Engine.
>
> This sounds like a bug in HTTP::Body, a patch would be great, thanks!
>
> -Andy
Patch attached. Hope it's okay.
--Tobias
--- MultiPart.pm.orig 2007-06-08 17:16:12.000000000 +0200
+++ MultiPart.pm 2007-06-08 17:18:42.000000000 +0200
@@ -273,7 +273,7 @@
if ($filename) {
- my $fh = File::Temp->new( UNLINK => 0 );
+ my $fh = File::Temp->new( UNLINK => 0, DIR => $self->{tmpdir} );
$part->{fh} = $fh;
$part->{tempname} = $fh->filename;
_______________________________________________
List: Catalyst@lists.rawmode.org
Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/catalyst@lists.rawmode.org/
Dev site: http://dev.catalyst.perl.org/
