[Catalyst] Catalyst::Request::Upload - uploadtmp

[Tobias Kremer]

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.

Thanks!

--Tobias