Feature/alternative autostart win

main/pom.xml

@@ -28,7 +28,7 @@
 		<cryptomator.jni.version>2.2.2</cryptomator.jni.version>
 		<cryptomator.fuse.version>1.2.3</cryptomator.fuse.version>
 		<cryptomator.dokany.version>1.1.15</cryptomator.dokany.version>
-		<cryptomator.webdav.version>1.0.11</cryptomator.webdav.version>
+		<cryptomator.webdav.version>1.0.12</cryptomator.webdav.version>
 
 		<!-- 3rd party dependencies -->
 		<javafx.version>14</javafx.version>

main/ui/src/main/java/org/cryptomator/ui/preferences/AutoStartModule.java

@@ -3,6 +3,7 @@
 import dagger.Module;
 import dagger.Provides;
 import org.apache.commons.lang3.SystemUtils;
+import org.cryptomator.common.Environment;
 import org.cryptomator.jni.MacFunctions;
 
 import java.util.Optional;
@@ -12,7 +13,7 @@ abstract class AutoStartModule {
 
 	@Provides
 	@PreferencesScoped
-	public static Optional<AutoStartStrategy> provideAutoStartStrategy(Optional<MacFunctions> macFunctions) {
+	public static Optional<AutoStartStrategy> provideAutoStartStrategy(Optional<MacFunctions> macFunctions, Environment env) {
 		if (SystemUtils.IS_OS_MAC_OSX && macFunctions.isPresent()) {
 			return Optional.of(new AutoStartMacStrategy(macFunctions.get()));
 		} else if (SystemUtils.IS_OS_WINDOWS) {

main/ui/src/main/java/org/cryptomator/ui/preferences/AutoStartWinStrategy.java

@@ -4,76 +4,185 @@
 import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.NoSuchFileException;
+import java.nio.file.Path;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.CompletionStage;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
-import java.util.stream.Collectors;
 
+/**
+ * OS specific class to check, en- and disable the auto start on Windows.
+ * <p>
+ * Two strategies are implemented for this feature, the first uses the registry and the second one the autostart folder.
+ * <p>
+ * The registry strategy checks/add/removes at the registry key {@value HKCU_AUTOSTART_KEY} an entry for Cryptomator.
+ * The folder strategy checks/add/removes at the location {@value WINDOWS_START_MENU_ENTRY}.
+ * <p>
+ * To check if the feature is active, both strategies are applied.
+ * To enable the feature, first the registry is tried and only on failure the autostart folder is used.
+ * To disable it, first it is determined by an internal state, which strategies must be used and in the second step those are executed.
+ *
+ * @apiNote This class is not thread safe, hence it should be avoided to call its methods simultaniously by different threads.
+ */
 class AutoStartWinStrategy implements AutoStartStrategy {
 
 	private static final Logger LOG = LoggerFactory.getLogger(AutoStartWinStrategy.class);
 	private static final String HKCU_AUTOSTART_KEY = "\"HKCU\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Run\"";
 	private static final String AUTOSTART_VALUE = "Cryptomator";
+	private static final String WINDOWS_START_MENU_ENTRY = "\\AppData\\Roaming\\Microsoft\\Windows\\Start Menu\\Programs\\Cryptomator.lnk";
+
 	private final String exePath;
 
+	private boolean activatedUsingFolder;
+	private boolean activatedUsingRegistry;
+
 	public AutoStartWinStrategy(String exePath) {
 		this.exePath = exePath;
+		this.activatedUsingFolder = false;
+		this.activatedUsingRegistry = false;
 	}
 
 	@Override
 	public CompletionStage<Boolean> isAutoStartEnabled() {
+		return isAutoStartEnabledUsingRegistry().thenCombine(isAutoStartEnabledUsingFolder(), (bReg, bFolder) -> bReg || bFolder);
+	}
+
+	private CompletableFuture<Boolean> isAutoStartEnabledUsingFolder() {
+		Path autoStartEntry = Path.of(System.getProperty("user.home") + WINDOWS_START_MENU_ENTRY);
+		this.activatedUsingFolder = Files.exists(autoStartEntry);
+		return CompletableFuture.completedFuture(activatedUsingFolder);
+	}
+
+	private CompletableFuture<Boolean> isAutoStartEnabledUsingRegistry() {
 		ProcessBuilder regQuery = new ProcessBuilder("reg", "query", HKCU_AUTOSTART_KEY, //
 				"/v", AUTOSTART_VALUE);
 		try {
 			Process proc = regQuery.start();
-			return proc.onExit().thenApply(p -> p.exitValue() == 0);
+			return proc.onExit().thenApply(p -> {
+				this.activatedUsingRegistry = p.exitValue() == 0;
+				return activatedUsingRegistry;
+			});
 		} catch (IOException e) {
-			LOG.warn("Failed to query {} from registry key {}", AUTOSTART_VALUE, HKCU_AUTOSTART_KEY);
+			LOG.debug("Failed to query {} from registry key {}", AUTOSTART_VALUE, HKCU_AUTOSTART_KEY);
 			return CompletableFuture.completedFuture(false);
 		}
 	}
 
 	@Override
 	public void enableAutoStart() throws TogglingAutoStartFailedException {
+		try {
+			enableAutoStartUsingRegistry().thenAccept((Void v) -> this.activatedUsingRegistry = true).exceptionallyCompose(e -> {
+				LOG.debug("Falling back to using autostart folder.");
+				return this.enableAutoStartUsingFolder();
+			}).thenAccept((Void v) -> this.activatedUsingFolder = true).get();
+		} catch (InterruptedException e) {
+			Thread.currentThread().interrupt();
+			throw new TogglingAutoStartFailedException("Execution of enabling auto start setting was interrupted.");
+		} catch (ExecutionException e) {
+			throw new TogglingAutoStartFailedException("Enabling auto start failed both using registry and auto start folder.");
+		}
+	}
+
+	private CompletableFuture<Void> enableAutoStartUsingRegistry() {
 		ProcessBuilder regAdd = new ProcessBuilder("reg", "add", HKCU_AUTOSTART_KEY, //
 				"/v", AUTOSTART_VALUE, //
 				"/t", "REG_SZ", //
 				"/d", "\"" + exePath + "\"", //
 				"/f");
-		String command = regAdd.command().stream().collect(Collectors.joining(" "));
 		try {
 			Process proc = regAdd.start();
-			boolean finishedInTime = waitForProcess(proc, 5, TimeUnit.SECONDS);
-			if (finishedInTime) {
+			boolean finishedInTime = waitForProcessOrCancel(proc, 5, TimeUnit.SECONDS);
+			if (finishedInTime && proc.exitValue() == 0) {
 				LOG.debug("Added {} to registry key {}.", AUTOSTART_VALUE, HKCU_AUTOSTART_KEY);
+				return CompletableFuture.completedFuture(null);
+			} else {
+				throw new IOException("Process existed with error code " + proc.exitValue());
+			}
+		} catch (IOException e) {
+			LOG.debug("Registry could not be edited to set auto start.", e);
+			return CompletableFuture.failedFuture(new SystemCommandException("Adding registry value failed."));
+		}
+	}
+
+	private CompletableFuture<Void> enableAutoStartUsingFolder() {
+		String autoStartFolderEntry = System.getProperty("user.home") + WINDOWS_START_MENU_ENTRY;
+		String createShortcutCommand = "$s=(New-Object -COM WScript.Shell).CreateShortcut('" + autoStartFolderEntry + "');$s.TargetPath='" + exePath + "';$s.Save();";
+		ProcessBuilder shortcutAdd = new ProcessBuilder("cmd", "/c", "Start powershell " + createShortcutCommand);
+		try {
+			Process proc = shortcutAdd.start();
+			boolean finishedInTime = waitForProcessOrCancel(proc, 5, TimeUnit.SECONDS);
+			if (finishedInTime && proc.exitValue() == 0) {
+				LOG.debug("Created file {} for auto start.", autoStartFolderEntry);
+				return CompletableFuture.completedFuture(null);
 			} else {
-				throw new TogglingAutoStartFailedException("Adding registry value failed.");
+				throw new IOException("Process existed with error code " + proc.exitValue());
 			}
 		} catch (IOException e) {
-			throw new TogglingAutoStartFailedException("Adding registry value failed. " + command, e);
+			LOG.debug("Adding entry to auto start folder failed.", e);
+			return CompletableFuture.failedFuture(new SystemCommandException("Adding entry to auto start folder failed."));
 		}
 	}
 
+
 	@Override
 	public void disableAutoStart() throws TogglingAutoStartFailedException {
+		if (activatedUsingRegistry) {
+			disableAutoStartUsingRegistry().whenComplete((voit, ex) -> {
+				if (ex == null) {
+					this.activatedUsingRegistry = false;
+				}
+			});
+		}
+
+		if (activatedUsingFolder) {
+			disableAutoStartUsingFolder().whenComplete((voit, ex) -> {
+				if (ex == null) {
+					this.activatedUsingFolder = false;
+				}
+			});
+		}
+
+		if (activatedUsingRegistry || activatedUsingFolder) {
+			throw new TogglingAutoStartFailedException("Disabling auto start failed both using registry and auto start folder.");
+		}
+	}
+
+	public CompletableFuture<Void> disableAutoStartUsingRegistry() {
 		ProcessBuilder regRemove = new ProcessBuilder("reg", "delete", HKCU_AUTOSTART_KEY, //
 				"/v", AUTOSTART_VALUE, //
 				"/f");
-		String command = regRemove.command().stream().collect(Collectors.joining(" "));
 		try {
 			Process proc = regRemove.start();
-			boolean finishedInTime = waitForProcess(proc, 5, TimeUnit.SECONDS);
-			if (finishedInTime) {
+			boolean finishedInTime = waitForProcessOrCancel(proc, 5, TimeUnit.SECONDS);
+			if (finishedInTime && proc.exitValue() == 0) {
 				LOG.debug("Removed {} from registry key {}.", AUTOSTART_VALUE, HKCU_AUTOSTART_KEY);
+				return CompletableFuture.completedFuture(null);
 			} else {
-				throw new TogglingAutoStartFailedException("Removing registry value failed.");
+				throw new IOException("Process existed with error code " + proc.exitValue());
 			}
 		} catch (IOException e) {
-			throw new TogglingAutoStartFailedException("Removing registry value failed. " + command, e);
+			LOG.debug("Registry could not be edited to remove auto start.", e);
+			return CompletableFuture.failedFuture(new SystemCommandException("Removing registry value failed."));
 		}
 	}
 
-	private static boolean waitForProcess(Process proc, int timeout, TimeUnit timeUnit) {
+	private CompletableFuture<Void> disableAutoStartUsingFolder() {
+		try {
+			Files.delete(Path.of(WINDOWS_START_MENU_ENTRY));
+			LOG.debug("Successfully deleted {}.", WINDOWS_START_MENU_ENTRY);
+			return CompletableFuture.completedFuture(null);
+		} catch (NoSuchFileException e) {
+			//that is also okay
+			return CompletableFuture.completedFuture(null);
+		} catch (IOException e) {
+			LOG.debug("Failed to delete entry from auto start folder.", e);
+			return CompletableFuture.failedFuture(e);
+		}
+	}
+
+	private static boolean waitForProcessOrCancel(Process proc, int timeout, TimeUnit timeUnit) {
 		boolean finishedInTime = false;
 		try {
 			finishedInTime = proc.waitFor(timeout, timeUnit);
@@ -88,4 +197,11 @@ private static boolean waitForProcess(Process proc, int timeout, TimeUnit timeUn
 		return finishedInTime;
 	}
 
+	private class SystemCommandException extends RuntimeException {
+
+		public SystemCommandException(String msg) {
+			super(msg);
+		}
+	}
+
 }