diff --git a/src/PVE/Auth/LDAP.pm b/src/PVE/Auth/LDAP.pm index 6a73ee8..f29d1ba 100644 --- a/src/PVE/Auth/LDAP.pm +++ b/src/PVE/Auth/LDAP.pm @@ -387,6 +387,7 @@ sub get_groups { } $ret->{$name} = { users => {} }; + $ret->{$name}->{comment} = $group->{description} if defined($group->{description}); foreach my $member (@{ $group->{members} }) { if (my $user = $dnmap->{ lc($member) }) { $ret->{$name}->{users}->{$user} = 1; diff --git a/src/test/realm_sync_test.pl b/src/test/realm_sync_test.pl index 10520db..e55b6cb 100644 --- a/src/test/realm_sync_test.pl +++ b/src/test/realm_sync_test.pl @@ -36,7 +36,7 @@ my $initialusercfg = { }, }, groups => { - 'group1-syncedrealm' => { users => {} }, + 'group1-syncedrealm' => { users => {}, comment => 'outdated comment' }, 'group2-syncedrealm' => { users => {} }, }, acl_root => { @@ -65,6 +65,7 @@ my $sync_response = { groups => [ { dn => 'dc=group1,dc=syncedrealm', + description => 'First group', members => [ 'uid=user1,dc=syncedrealm', ], @@ -176,6 +177,7 @@ my $tests = [ users => { 'user1@syncedrealm' => 1, }, + comment => 'First group', }, 'group2-syncedrealm' => { users => {} }, 'group3-syncedrealm' => { users => {} }, @@ -216,6 +218,7 @@ my $tests = [ users => { 'user1@syncedrealm' => 1, }, + comment => 'First group', }, 'group3-syncedrealm' => { users => {} }, }, @@ -260,6 +263,7 @@ my $tests = [ users => { 'user1@syncedrealm' => 1, }, + comment => 'First group', }, 'group2-syncedrealm' => { users => {} }, 'group3-syncedrealm' => { users => {} }, @@ -298,6 +302,7 @@ my $tests = [ users => { 'user1@syncedrealm' => 1, }, + comment => 'First group', }, 'group3-syncedrealm' => { users => {} }, }, @@ -336,6 +341,7 @@ my $tests = [ users => { 'user1@syncedrealm' => 1, }, + comment => 'First group', }, 'group3-syncedrealm' => { users => {} }, },