adjust code after forth review

src/keyutils_persistent.rs

@@ -38,8 +38,9 @@ impl CredentialApi for KeyutilsPersistentCredential {
     }
 
     fn get_password(&self) -> Result<String> {
-        if let Ok(password) = self.keyutils.get_password() {
-            return Ok(password);
+        match self.keyutils.get_password() {
+            Ok(password) => return Ok(password),
+            Err(err) => debug!("cannot get password from keyutils, trying from secret service"),
         }
 
         let password = self.ss.get_password().map_err(ambigous_to_no_entry)?;
@@ -49,8 +50,9 @@ impl CredentialApi for KeyutilsPersistentCredential {
     }
 
     fn get_secret(&self) -> Result<Vec<u8>> {
-        if let Ok(secret) = self.keyutils.get_secret() {
-            return Ok(secret);
+        match self.keyutils.get_secret() {
+            Ok(secret) => return Ok(secret),
+            Err(err) => debug!("cannot get secret from keyutils, trying from secret service"),
         }
 
         let secret = self.ss.get_secret().map_err(ambigous_to_no_entry)?;
@@ -63,6 +65,7 @@ impl CredentialApi for KeyutilsPersistentCredential {
         if let Err(err) = self.keyutils.delete_credential() {
             debug!("cannot delete keyutils credential: {err}");
         }
+
         self.ss.delete_credential()
     }
 

src/lib.rs

@@ -239,11 +239,18 @@ pub mod keyutils_persistent;
 pub use keyutils_persistent as default;
 
 // fallback to mock if neither keyutils nor secret service is available
-#[cfg(all(
-    any(target_os = "linux", target_os = "freebsd", target_os = "openbsd"),
-    not(feature = "linux-native"),
-    not(feature = "sync-secret-service"),
-    not(feature = "async-secret-service"),
+#[cfg(any(
+    all(
+        target_os = "linux",
+        not(feature = "linux-native"),
+        not(feature = "sync-secret-service"),
+        not(feature = "async-secret-service"),
+    ),
+    all(
+        any(target_os = "freebsd", target_os = "openbsd"),
+        not(feature = "sync-secret-service"),
+        not(feature = "async-secret-service"),
+    )
 ))]
 pub use mock as default;