Merge pull request #1 from swiesend/develop

Use secret-service 1.5.0. 

Closes #2 .

Author: Armin Schrenk

pom.xml

@@ -39,7 +39,7 @@
 
 		<!-- runtime dependencies -->
 		<api.version>0.1.3</api.version>
-		<secret-service.version>1.2.1</secret-service.version>
+		<secret-service.version>1.5.0</secret-service.version>
 		<kdewallet.version>1.1.1</kdewallet.version>
 		<guava.version>30.0-jre</guava.version>
 		<slf4j.version>1.7.30</slf4j.version>

src/main/java/org/cryptomator/linux/keychain/SecretServiceKeychainAccess.java

@@ -5,6 +5,7 @@
 import org.freedesktop.secret.simple.SimpleCollection;
 
 import java.io.IOException;
+import java.security.AccessControlException;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -15,24 +16,19 @@ public class SecretServiceKeychainAccess implements KeychainAccessProvider {
 
 	@Override
 	public boolean isSupported() {
-		try (@SuppressWarnings("unused") SimpleCollection keyring = new SimpleCollection()) {
-			// seems like we're able to access the keyring.
-			return true;
-		} catch (IOException | ExceptionInInitializerError | RuntimeException e) {
-			return false;
-		}
+		return SimpleCollection.isAvailable();
 	}
 
 	@Override
 	public void storePassphrase(String key, CharSequence passphrase) throws KeychainAccessException {
 		try (SimpleCollection keyring = new SimpleCollection()) {
 			List<String> list = keyring.getItems(createAttributes(key));
-			if (list == null) {
+			if (list == null || list.isEmpty()) {
 				keyring.createItem(LABEL_FOR_SECRET_IN_KEYRING, passphrase, createAttributes(key));
 			} else {
 				changePassphrase(key, passphrase);
 			}
-		} catch (IOException e) {
+		} catch (IOException | AccessControlException e) {
 			throw new KeychainAccessException("Storing password failed.", e);
 		}
 	}
@@ -41,12 +37,12 @@ public void storePassphrase(String key, CharSequence passphrase) throws Keychain
 	public char[] loadPassphrase(String key) throws KeychainAccessException {
 		try (SimpleCollection keyring = new SimpleCollection()) {
 			List<String> list = keyring.getItems(createAttributes(key));
-			if (list != null) {
+			if (list != null && !list.isEmpty()) {
 				return keyring.getSecret(list.get(0));
 			} else {
 				return null;
 			}
-		} catch (IOException e) {
+		} catch (IOException | AccessControlException e) {
 			throw new KeychainAccessException("Loading password failed.", e);
 		}
 	}
@@ -55,10 +51,10 @@ public char[] loadPassphrase(String key) throws KeychainAccessException {
 	public void deletePassphrase(String key) throws KeychainAccessException {
 		try (SimpleCollection keyring = new SimpleCollection()) {
 			List<String> list = keyring.getItems(createAttributes(key));
-			if (list != null) {
+			if (list != null && !list.isEmpty()) {
 				keyring.deleteItem(list.get(0));
 			}
-		} catch (IOException e) {
+		} catch (IOException | AccessControlException e) {
 			throw new KeychainAccessException("Deleting password failed.", e);
 		}
 	}
@@ -67,15 +63,16 @@ public void deletePassphrase(String key) throws KeychainAccessException {
 	public void changePassphrase(String key, CharSequence passphrase) throws KeychainAccessException {
 		try (SimpleCollection keyring = new SimpleCollection()) {
 			List<String> list = keyring.getItems(createAttributes(key));
-			if (list != null) {
+			if (list != null && !list.isEmpty()) {
 				keyring.updateItem(list.get(0), LABEL_FOR_SECRET_IN_KEYRING, passphrase, createAttributes(key));
 			}
-		} catch (IOException e) {
+		} catch (IOException | AccessControlException e) {
 			throw new KeychainAccessException("Changing password failed.", e);
 		}
 	}
 
 	private Map<String, String> createAttributes(String key) {
 		return Map.of("Vault", key);
 	}
+
 }