[PATCH datacenter-manager v4 6/6] fix #6914: cli-admin: add option to delete token from remote

[Shan Shaji <s.shaji@proxmox.com>]

The `delete-token` option will be `true` by default. If not specified
the token will be deleted automatically. Inorder to skip the token
deletion, set the `delete-token` to `false`.

Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
---
 changes since v3: Thanks @Lukas
 - updated commit message.
 - drop `type` and set the `default` prop.
 
 cli/admin/src/remotes.rs | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/cli/admin/src/remotes.rs b/cli/admin/src/remotes.rs
index edd3fd8..c2930b7 100644
--- a/cli/admin/src/remotes.rs
+++ b/cli/admin/src/remotes.rs
@@ -217,16 +217,25 @@ fn update_remote(
     input: {
         properties: {
             id: { schema: REMOTE_ID_SCHEMA },
+            "delete-token": {
+                optional: true,
+                default: true,
+                description: "If set to false, token deletion on the remote is skipped."
+            }
         }
     }
 )]
-/// Add a new remote.
-fn remove_remote(id: String, rpcenv: &mut dyn RpcEnvironment) -> Result<(), Error> {
-    let param = json!({ "id": id });
+/// Remove a remote.
+async fn remove_remote(
+    id: String,
+    delete_token: bool,
+    rpcenv: &mut dyn RpcEnvironment,
+) -> Result<(), Error> {
+    let param = json!({ "id": id, "delete-token": delete_token });
 
     let info = &dc_api::remotes::API_METHOD_REMOVE_REMOTE;
     match info.handler {
-        ApiHandler::Sync(handler) => (handler)(param, info, rpcenv).map(drop),
+        ApiHandler::Async(handler) => (handler)(param, info, rpcenv).await.map(drop),
         _ => unreachable!(),
     }
 }
-- 
2.47.3