clippy: allow api functions with many arguments
some of those can be reduced/cleaned up when we have updater support in the api macro. Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
parent
9c26a3d61a
commit
367c0ff7c6
|
@ -165,6 +165,7 @@ pub fn read_acl(
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update Access Control List (ACLs).
|
/// Update Access Control List (ACLs).
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_acl(
|
pub fn update_acl(
|
||||||
path: String,
|
path: String,
|
||||||
role: String,
|
role: String,
|
||||||
|
|
|
@ -421,6 +421,7 @@ impl TfaUpdateInfo {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Add a TFA entry to the user.
|
/// Add a TFA entry to the user.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
fn add_tfa_entry(
|
fn add_tfa_entry(
|
||||||
userid: Userid,
|
userid: Userid,
|
||||||
description: Option<String>,
|
description: Option<String>,
|
||||||
|
|
|
@ -353,6 +353,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update user configuration.
|
/// Update user configuration.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_user(
|
pub fn update_user(
|
||||||
userid: Userid,
|
userid: Userid,
|
||||||
comment: Option<String>,
|
comment: Option<String>,
|
||||||
|
|
|
@ -1293,8 +1293,6 @@ pub fn catalog(
|
||||||
backup_id: String,
|
backup_id: String,
|
||||||
backup_time: i64,
|
backup_time: i64,
|
||||||
filepath: String,
|
filepath: String,
|
||||||
_param: Value,
|
|
||||||
_info: &ApiMethod,
|
|
||||||
rpcenv: &mut dyn RpcEnvironment,
|
rpcenv: &mut dyn RpcEnvironment,
|
||||||
) -> Result<Value, Error> {
|
) -> Result<Value, Error> {
|
||||||
let datastore = DataStore::lookup_datastore(&store)?;
|
let datastore = DataStore::lookup_datastore(&store)?;
|
||||||
|
|
|
@ -277,6 +277,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update datastore config.
|
/// Update datastore config.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_datastore(
|
pub fn update_datastore(
|
||||||
name: String,
|
name: String,
|
||||||
comment: Option<String>,
|
comment: Option<String>,
|
||||||
|
|
|
@ -203,6 +203,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update remote configuration.
|
/// Update remote configuration.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_remote(
|
pub fn update_remote(
|
||||||
name: String,
|
name: String,
|
||||||
comment: Option<String>,
|
comment: Option<String>,
|
||||||
|
|
|
@ -279,6 +279,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update sync job config.
|
/// Update sync job config.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_sync_job(
|
pub fn update_sync_job(
|
||||||
id: String,
|
id: String,
|
||||||
store: Option<String>,
|
store: Option<String>,
|
||||||
|
|
|
@ -215,6 +215,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update verification job config.
|
/// Update verification job config.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_verification_job(
|
pub fn update_verification_job(
|
||||||
id: String,
|
id: String,
|
||||||
store: Option<String>,
|
store: Option<String>,
|
||||||
|
|
|
@ -213,6 +213,7 @@ pub fn read_interface(iface: String) -> Result<Value, Error> {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Create network interface configuration.
|
/// Create network interface configuration.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn create_interface(
|
pub fn create_interface(
|
||||||
iface: String,
|
iface: String,
|
||||||
autostart: Option<bool>,
|
autostart: Option<bool>,
|
||||||
|
@ -477,6 +478,7 @@ pub enum DeletableProperty {
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Update network interface config.
|
/// Update network interface config.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn update_interface(
|
pub fn update_interface(
|
||||||
iface: String,
|
iface: String,
|
||||||
autostart: Option<bool>,
|
autostart: Option<bool>,
|
||||||
|
|
|
@ -425,6 +425,7 @@ fn stop_task(
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// List tasks.
|
/// List tasks.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub fn list_tasks(
|
pub fn list_tasks(
|
||||||
start: u64,
|
start: u64,
|
||||||
limit: u64,
|
limit: u64,
|
||||||
|
|
|
@ -112,6 +112,7 @@ fn extract_archive_from_reader<R: std::io::Read>(
|
||||||
},
|
},
|
||||||
)]
|
)]
|
||||||
/// Extract an archive.
|
/// Extract an archive.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
fn extract_archive(
|
fn extract_archive(
|
||||||
archive: String,
|
archive: String,
|
||||||
pattern: Option<Vec<String>>,
|
pattern: Option<Vec<String>>,
|
||||||
|
|
|
@ -57,6 +57,8 @@ impl BackupWriter {
|
||||||
Arc::new(Self { h2, abort, crypt_config, verbose })
|
Arc::new(Self { h2, abort, crypt_config, verbose })
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// FIXME: extract into (flattened) parameter struct?
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub async fn start(
|
pub async fn start(
|
||||||
client: HttpClient,
|
client: HttpClient,
|
||||||
crypt_config: Option<Arc<CryptConfig>>,
|
crypt_config: Option<Arc<CryptConfig>>,
|
||||||
|
|
Loading…
Reference in New Issue