proxmox-mirrors/pve-access-control
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-access-control synced 2025-10-04 14:18:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

jobs: realm sync: fix scheduled LDAP syncs not applying attributes correctly

Browse Source 
This was reported by a user in the forum [0].

The cause was that the user-* standard options were not registered
when the sync was called from the scheduler, resulting in the
following error:

  pvescheduler[2849]: skipping attribute mapping 'cn'->'comment' for user 'test@samba0' - no such standard option 'user-comment'

Fix this by simply importing the PVE::API2::User module, thus ensuring
the options get registered.

[0] https://forum.proxmox.com/threads/ldap-integration-comment-email-first-name-lastname.143490/

Fixes: cb93636 ("LDAP sync: improve validation of synced attributes")
Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>
Reviewed-by: Fiona Ebner <f.ebner@proxmox.com>
Tested-by: Fiona Ebner <f.ebner@proxmox.com>
This commit is contained in:
Christoph Heiss 2024-03-19 14:48:43 +01:00 committed by Thomas Lamprecht
parent 5bcf553e3a
commit bb34ca534e
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/PVE/Jobs/RealmSync.pm
View File

@ -13,6 +13,9 @@ use PVE::Tools ();
use PVE::API2::Domains (); use PVE::API2::Domains ();
# load user-* standard options
use PVE::API2::User ();
use base qw(PVE::Job::Registry); use base qw(PVE::Job::Registry);
sub type { sub type {