[
https://issues.apache.org/jira/browse/KARAF-4475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15228542#comment-15228542
]
Fabian Lange commented on KARAF-4475:
-------------------------------------
Actually this class does a couple more "state" changing calls, so it should not
be used as config parser
> 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
> Reporter: Fabian Lange
>
> 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)
