mirror of
https://github.com/ViViDboarder/bitwarden_rs_ldap.git
synced 2024-11-24 04:06:26 +00:00
Make checking for existing users case insensitive
Email addresses are not case case sensitive, so when we check for an existing user, we should make sure we do it in a way that is not sensitive to case. Fixes #8
This commit is contained in:
parent
42489d39e1
commit
7f35c2a989
@ -42,7 +42,7 @@ fn get_existing_users(client: &mut bw_admin::Client) -> Result<HashSet<String>,
|
|||||||
let all_users = client.users()?;
|
let all_users = client.users()?;
|
||||||
let mut user_emails = HashSet::with_capacity(all_users.len());
|
let mut user_emails = HashSet::with_capacity(all_users.len());
|
||||||
for user in all_users {
|
for user in all_users {
|
||||||
user_emails.insert(user.get_email());
|
user_emails.insert(user.get_email().to_lowercase());
|
||||||
if user.is_disabled() {
|
if user.is_disabled() {
|
||||||
println!(
|
println!(
|
||||||
"Existing disabled user found with email: {}",
|
"Existing disabled user found with email: {}",
|
||||||
@ -124,7 +124,7 @@ fn invite_from_ldap(
|
|||||||
.get(mail_field.as_str())
|
.get(mail_field.as_str())
|
||||||
.and_then(|l| (l.first()))
|
.and_then(|l| (l.first()))
|
||||||
{
|
{
|
||||||
if existing_users.contains(user_email) {
|
if existing_users.contains(&user_email.to_lowercase()) {
|
||||||
println!("User with email already exists: {}", user_email);
|
println!("User with email already exists: {}", user_email);
|
||||||
} else {
|
} else {
|
||||||
println!("Try to invite user: {}", user_email);
|
println!("Try to invite user: {}", user_email);
|
||||||
|
Loading…
Reference in New Issue
Block a user