Authenticator refactor (#486)

* Tidy sync flags.

* Tweak doc comments.

* Prune obsolete code.

* Remove NO_SYNC flag from authenticator.

* Support options in create_folder().

* Rename methods in VaultAccess.

* Tidy functions for flag access and doc comments.

* Support setting vault flags in VaultAccess.

* Respect SetVaultFlags event in folder reducer.

* Add test spec for set_vault_flags().

* Expose update_folder_flags() in Account.

* Support remove_local_folder().

To delete a folder, not apply the AccountEvent::DeleteFolder event to
the account logs so the folder will not be deleted from other devices
and the network implementation does not need to sync.

This is for folders which need special handling such as an authenticator
folder.

* Add initial NO_SYNC test spec.

* Test for is_sync_disabled() when building UpdateSet.

* Test for is_sync_disabled() when building sync diff.

* Add test spec for NO_SYNC flag.

* Bump patch version.

Author: tmpfs

Cargo.lock

@@ -158,7 +158,9 @@ async fn simulate_session(
     // Create a new folder
     let FolderCreate {
         folder: new_folder, ..
-    } = account.create_folder("New folder".to_string()).await?;
+    } = account
+        .create_folder("New folder".to_string(), Default::default())
+        .await?;
     // Rename the folder
     account
         .rename_folder(&new_folder, "New name".to_string())

crates/integration_tests/tests/audit_trail/client.rs

@@ -45,7 +45,9 @@ async fn diff_merge_folder_create() -> Result<()> {
     // Create a new folder
     let FolderCreate {
         folder: summary, ..
-    } = local.create_folder("new_folder".to_owned()).await?;
+    } = local
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
 
     assert_ne!(local.sync_status().await?, remote.sync_status().await?);
 

crates/integration_tests/tests/diff_merge/folder_create.rs

@@ -42,7 +42,9 @@ async fn diff_merge_folder_delete() -> Result<()> {
     // Create a new folder
     let FolderCreate {
         folder: summary, ..
-    } = local.create_folder("new_folder".to_owned()).await?;
+    } = local
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
 
     // Delete the folder
     local.delete_folder(&summary).await?;

crates/integration_tests/tests/diff_merge/folder_delete.rs

@@ -52,7 +52,9 @@ async fn diff_merge_folder_import() -> Result<()> {
     temp.sign_in(&key).await?;
     let FolderCreate {
         folder: summary, ..
-    } = temp.create_folder("new_folder".to_owned()).await?;
+    } = temp
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
     let (folder_password, _) = generate_passphrase()?;
     let folder_key: AccessKey = folder_password.into();
     let exported = data_dir_export.join("exported.vault");

crates/integration_tests/tests/diff_merge/folder_import.rs

@@ -52,7 +52,9 @@ async fn diff_merge_secret_move() -> Result<()> {
     // Create a new folder
     let FolderCreate {
         folder: summary, ..
-    } = local.create_folder("new_folder".to_owned()).await?;
+    } = local
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
 
     // Move the secret
     let SecretMove { id: new_id, .. } = local

crates/integration_tests/tests/diff_merge/secret_move.rs

@@ -33,8 +33,9 @@ async fn event_log_account() -> Result<()> {
     // Create a folder
     let commit = event_log.tree().last_commit();
     let folder_name = "folder_name";
-    let FolderCreate { folder, .. } =
-        account.create_folder(folder_name.to_string()).await?;
+    let FolderCreate { folder, .. } = account
+        .create_folder(folder_name.to_string(), Default::default())
+        .await?;
 
     let event = last_log_event(&mut event_log, commit.as_ref()).await?;
     assert!(matches!(event, Some(AccountEvent::CreateFolder(_, _))));

crates/integration_tests/tests/event_log/account_events.rs

@@ -29,8 +29,9 @@ async fn event_log_file() -> Result<()> {
 
     // Create a folder so we can move the secret
     let folder_name = "folder_name";
-    let FolderCreate { folder, .. } =
-        account.create_folder(folder_name.to_string()).await?;
+    let FolderCreate { folder, .. } = account
+        .create_folder(folder_name.to_string(), Default::default())
+        .await?;
 
     // Create an external file secret
     let (meta, secret, _file_path) = mock::file_text_secret()?;

crates/integration_tests/tests/event_log/file_events.rs

@@ -26,7 +26,10 @@ async fn file_transfers_abort_move() -> Result<()> {
     let FolderCreate {
         folder: destination,
         ..
-    } = device.owner.create_folder("new_folder".to_owned()).await?;
+    } = device
+        .owner
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
 
     // Create an external file secret
     let (secret_id, _, _, file_name) =

crates/integration_tests/tests/file_transfers/abort_move.rs

@@ -166,7 +166,10 @@ async fn file_transfers_attach_move() -> Result<()> {
     let FolderCreate {
         folder: destination,
         ..
-    } = device.owner.create_folder("new_folder".to_owned()).await?;
+    } = device
+        .owner
+        .create_folder("new_folder".to_owned(), Default::default())
+        .await?;
 
     // Moving the secret also needs to move the files
     let SecretMove { id: secret_id, .. } = device