Skip to content

Commit 6da85a6

Browse files
authored
Merge pull request #14 from SuperSandro2000/dedup
Dedup group members
2 parents 2c4dca6 + 9c79afc commit 6da85a6

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

rust/userborn/src/main.rs

+4-2
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,9 @@ fn update_users_and_groups(
9898
) {
9999
for group_config in &config.groups {
100100
if let Some(existing_entry) = group_db.get_mut(&group_config.name) {
101-
existing_entry.update(group_config.members.clone());
101+
let mut members = group_config.members.clone();
102+
members.dedup();
103+
existing_entry.update(members);
102104
} else if let Err(e) = create_group(group_config, group_db) {
103105
log::error!("Failed to create group {}: {e:#}", group_config.name);
104106
};
@@ -401,7 +403,7 @@ mod tests {
401403

402404
#[test]
403405
fn update_users_and_groups_across_generations() -> Result<()> {
404-
// Explitly set this because the expected values depend on this.
406+
// Explicitly set this because the expected values depend on this.
405407
std::env::set_var("USERBORN_NO_LOGIN_PATH", NO_LOGIN_FALLBACK);
406408

407409
let mut group_db = Group::default();

0 commit comments

Comments
 (0)