Merge branch 'main' into 'main'
fix : clippy See merge request compsoc1/skynet/ldap/backend!21
This commit is contained in:
commit
100b0d6552
3 changed files with 3 additions and 3 deletions
|
@ -283,7 +283,7 @@ pub fn uid_to_dn(uid: &str) -> String {
|
|||
format!("uid={},ou=users,dc=skynet,dc=ie", uid)
|
||||
}
|
||||
|
||||
pub async fn update_group(config: &Config, group: &str, users: &Vec<String>, replace: bool) -> tide::Result<()> {
|
||||
pub async fn update_group(config: &Config, group: &str, users: &[String], replace: bool) -> tide::Result<()> {
|
||||
if users.is_empty() {
|
||||
return Ok(());
|
||||
}
|
||||
|
|
|
@ -421,7 +421,7 @@ pub mod account {
|
|||
ldap.extended(tmp).unwrap();
|
||||
|
||||
// user is already verified by being an active member on wolves
|
||||
if let Err(e) = update_group(config, "skynet-users", &vec![username.to_string()], false).await {
|
||||
if let Err(e) = update_group(config, "skynet-users", &[username.to_string()], false).await {
|
||||
println!("Couldnt add {} to skynet-users: {:?}", username, e)
|
||||
}
|
||||
|
||||
|
|
|
@ -87,7 +87,7 @@ async fn activate_group(db: &Pool<Sqlite>, config: &Config, user: &str, mail: &s
|
|||
// check if user has this mail in teh wolves db
|
||||
if !get_wolves_mail(db, mail).await.is_empty() {
|
||||
// if so then activate
|
||||
if let Err(e) = update_group(config, "skynet-users", &vec![user.to_string()], false).await {
|
||||
if let Err(e) = update_group(config, "skynet-users", &[user.to_string()], false).await {
|
||||
println!("Couldnt add {} to skynet-users: {:?}", user, e)
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue