Fabian Lange created KARAF-4475:
-----------------------------------

             Summary: 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)

Reply via email to