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
>>
>>
>

Reply via email to