[jira] [Resolved] (KARAF-4475) Performing status check will wipe cache if karaf.clean.all/karaf.clean.cache is set

["Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/KARAF-4475?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Onofré resolved KARAF-4475.
-----------------------------------------
    Resolution: Fixed

> Performing status check will wipe cache if karaf.clean.all/karaf.clean.cache is set
> -----------------------------------------------------------------------------------
>
>                 Key: KARAF-4475
>                 URL: https://issues.apache.org/jira/browse/KARAF-4475
>             Project: Karaf
>          Issue Type: Bug
>          Components: karaf-core
>            Reporter: Fabian Lange
>            Assignee: Jean-Baptiste Onofré
>             Fix For: 4.1.0, 4.0.5
>
>
> Whenever somebody does "new ConfigProperties()" the cache files are deleted:
> https://github.com/apache/karaf/blob/master/main/src/main/java/org/apache/karaf/main/ConfigProperties.java#L181
> This is problematic, as with a running karaf instance, somebody who will invoke the status script, will wipe the cache directory of the running instance, resulting in various problems
> Both: Status and Stop are guilty of doing that.
> https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Status.java#L38
> https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Stop.java#L42
> I propose to move the deletion logic to Main.launch()
> https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Main.java#L230



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)