Hi, This just seems like another thing to configure. We should just use java.io.tmpdir to store temporary files.
Justin On Mon, Jan 23, 2012 at 3:56 AM, Felix Meschberger <fmesc...@adobe.com> wrote: > Hi, > > While I am basically ok to use the Jackrabbit Defaults, another old > discussion pops up again: Generic temporary file handling in Sling. > > We currently have a number of places, where temporary files are created and > used. For example the Commons FileUpload library in the Engine or the > Jackrabbit based WebDAV bundles. > > I wonder, whether we should provide a service which has the following: > > * Provides access to a sling-wide temporary location (configurable, > default: ${sling.home}/tmp) > * Provides functionaliy to manage tmp files > * (For completeness) provides a method to create tmp files (based > on File.createTempFile using the configured tmp location) > > This way, we could configure the FileUpload library and the WebDAV bundles > accordingly. > > WDYT ? > > Regards > Felix > > Am 19.01.2012 um 20:24 schrieb Jukka Zitting (Commented) (JIRA): > >> >> [ >> https://issues.apache.org/jira/browse/SLING-2215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13189313#comment-13189313 >> ] >> >> Jukka Zitting commented on SLING-2215: >> -------------------------------------- >> >> Sorry, the SLING-2301 change did indeed touch this bundle, even though it >> mentions the simple webdav bundle instead of the davex bundle. >> >> Anyway, there's still something wrong as the jackrabbit/tmp directory gets >> created when I start the davex bundle. I'll investigate. >> >>> DavEx servlet creates a jackrabbit/tmp directory >>> ------------------------------------------------ >>> >>> Key: SLING-2215 >>> URL: https://issues.apache.org/jira/browse/SLING-2215 >>> Project: Sling >>> Issue Type: Bug >>> Affects Versions: JCR DavEx 1.0.0 >>> Reporter: Jukka Zitting >>> Priority: Minor >>> Fix For: JCR DavEx 1.1.0 >>> >>> Attachments: >>> 0001-SLING-2215-DavEx-servlet-creates-a-jackrabbit-tmp-di.patch >>> >>> >>> The DavEx servlet bundle added in SLING-1393 doesn't set the "home" and >>> "temp-directory" init parameters of the JcrRemotingServlet base class it >>> uses from Jackrabbit. As a result Jackrabbit defaults to "jackrabbit/tmp" >>> as the location >>> for temporary files, and creates such a directory under the current working >>> directory. >>> This problem is solved in the latest Jackrabbit trunk as of JCR-3029 where >>> the default location for temporary files was set >>> to the standard one defined by the java.io.tmpdir system property, so I >>> guess the easiest way to solve this is simply to >>> upgrade the dependency in Sling once the Jackrabbit 2.3.0 release is >>> available. >> >> -- >> This message is automatically generated by JIRA. >> If you think it was sent incorrectly, please contact your JIRA >> administrators: >> https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa >> For more information on JIRA, see: http://www.atlassian.com/software/jira >> >> >