proxmox-mirrors/proxmox-backup
1
0
Fork 1
mirror of https://git.proxmox.com/git/proxmox-backup synced 2025-10-04 17:05:37 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

realm sync: replace formatted .context() calls

Browse Source 
with .map_err/.ok_or_else - since the formatting should not
happen in the non-error case

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
This commit is contained in:
Wolfgang Bumiller 2023-03-27 11:51:17 +02:00
parent 750704400a
commit 5f0965edba
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/server/realm_sync_job.rs
View File

@ -1,4 +1,4 @@
use anyhow::{bail, Context, Error};
use anyhow::{bail, format_err, Context, Error};
use pbs_config::{acl::AclTree, token_shadow, BackupLockGuard};
use proxmox_lang::try_block;
use proxmox_ldap::{Config, Connection, SearchParameters, SearchResult};
@ -165,9 +165,11 @@ impl LdapRealmSyncJob {
let username = result
.attributes
.get(user_id_attribute)
.context(format!(
"userid attribute `{user_id_attribute}` not in LDAP search result"
))?
.ok_or_else(|| {
format_err!(
"userid attribute `{user_id_attribute}` not in LDAP search result"
)
})?
.get(0)
.context("userid attribute array is empty")?
.clone();
@ -176,7 +178,7 @@ impl LdapRealmSyncJob {
let userid: Userid = username
.parse()
.context(format!("could not parse username `{username}`"))?;
.map_err(|err| format_err!("could not parse username `{username}` - {err}"))?;
retrieved_users.insert(userid.clone());
self.create_or_update_user(user_config, &userid, result)?;