mirrors/cryptomator
1
0
Fork 0
mirror of https://github.com/cryptomator/cryptomator.git synced 2026-05-14 08:41:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Implement logic to overwrite system properties set in app internal config file

Browse Source
This commit is contained in:
Armin Schrenk
2026-01-13 13:14:40 +01:00
parent 11c86f7287
commit b23bd0b27a
1 changed files with 36 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
src/main/java/org/cryptomator/launcher/AdminPropertiesSetter.java
View File

@@ -11,28 +11,58 @@ import java.nio.file.Path;
import java.nio.file.StandardOpenOption;
import java.util.Properties;
/**
* Class to change JVM system properties according to an external properties file
* <p>
* The app starts with a set of predefined properties (defined in [APPDIR]/../Cryptomator.cfg).
* That file is difficult to edit and edits do not persist over updates.
* To allow overwriting these properties, the method {@link #adjustSystemProperties()} loads the properties file {@value PROP_FILENAME} from an OS-dependent location and add the contained properties to the {@link System} properties.
* The predefined location are:
* <ul>
* <li>Linux - {@value LINUX_DIR }</li>
* <li>macOS - {@value MAC_DIR }</li>
* <li>Windows - {@value WIN_DIR }</li>
* </ul>
* </p>
*
* @see System#getProperties()
*/
class AdminPropertiesSetter {
private static final Logger LOG = LoggerFactory.getLogger(AdminPropertiesSetter.class);
private static final String LINUX_DIR = "/etc/cryptmator";
private static final String MAC_DIR = "/Library/Application Support/Cryptomator";
private static final String WIN_DIR = "%PROGRAMDATA%\\Cryptomator";
private static final String PROP_FILENAME = "config.properties";
static {
final Path adminDir;
if (SystemUtils.IS_OS_WINDOWS) {
adminDir = Path.of(System.getenv().getOrDefault("ProgramData", "C:\\ProgramData"), "Cryptomator");
} else if (SystemUtils.IS_OS_MAC) {
adminDir = Path.of("/Library/Application Support/Cryptomator");
adminDir = Path.of(MAC_DIR);
} else { //LINUX
adminDir = Path.of("/etc/cryptmator");
adminDir = Path.of(LINUX_DIR);
}
ADMIN_PROPERTIES_DIR = adminDir;
ADMIN_PROPERTIES_FILE = adminDir.resolve(PROP_FILENAME);
}
private static final Path ADMIN_PROPERTIES_DIR;
private static final Path ADMIN_PROPERTIES_FILE = ADMIN_PROPERTIES_DIR.resolve("config.properties");
private static final Path ADMIN_PROPERTIES_FILE;
/**
* Adjusts the system properties by loading administrative properties from a predefined file location.
* <p>
* If the file exists and is a valid properties file, its properties will overwrite the existing system properties.
* <p>
* <em>WARNING:</em> This method modifies the global system properties and should be used with caution. Overwriting some properties has no effect, because they are read only once from the original config during JVM startup.
*
* @return The adjusted system properties.
*/
static Properties adjustSystemProperties() {
var systemProps = System.getProperties();
var adminProps = loadAdminProperties(ADMIN_PROPERTIES_FILE);
for (var key : adminProps.stringPropertyNames()) {
var value = adminProps.getProperty(key);
LOG.info("Overwriting {} with value {} from admin properties.", key, value);
@@ -47,6 +77,7 @@ class AdminPropertiesSetter {
adminProps.load(Files.newInputStream(adminPropertiesPath, StandardOpenOption.READ));
} catch (NoSuchFileException _) {
//NO-OP
LOG.debug("No admin properties found at {}.", adminPropertiesPath);
} catch (IOException | IllegalArgumentException e) {
LOG.warn("Failed to read administrative properties from {}. Returning empty properties.", adminPropertiesPath, e);
}