diff options
| author | Stanislav Laznicka <slaznick@redhat.com> | 2016-06-07 13:51:46 +0200 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2016-06-29 10:44:30 +0200 |
| commit | 427bbf6c0d61cf52f14a9f2606143f994340ec83 (patch) | |
| tree | 6370cda955d6d4d19b18a79426bba4a6b532b66e /ipaserver | |
| parent | 30d054a573e08aedecdc211d50c9dd8ebd2777ea (diff) | |
| download | freeipa-427bbf6c0d61cf52f14a9f2606143f994340ec83.tar.gz freeipa-427bbf6c0d61cf52f14a9f2606143f994340ec83.tar.xz freeipa-427bbf6c0d61cf52f14a9f2606143f994340ec83.zip | |
The LDAP*ReverseMember shouldn't imply --all is always specified
The LDAP*ReverseMember methods would always return the whole LDAP
object even though --all is not specified.
Also had to fix some tests as objectClass will not be returned by
default now.
https://fedorahosted.org/freeipa/ticket/5892
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
Diffstat (limited to 'ipaserver')
| -rw-r--r-- | ipaserver/plugins/baseldap.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipaserver/plugins/baseldap.py b/ipaserver/plugins/baseldap.py index 62b726da1..c35f660c7 100644 --- a/ipaserver/plugins/baseldap.py +++ b/ipaserver/plugins/baseldap.py @@ -2159,7 +2159,7 @@ class LDAPAddReverseMember(LDAPModReverseMember): failed['member'][self.reverse_attr].append((attr, unicode(e))) # Update the member data. - entry_attrs = ldap.get_entry(dn, ['*']) + entry_attrs = ldap.get_entry(dn, attrs_list) self.obj.convert_attribute_members(entry_attrs, *keys, **options) for callback in self.get_callbacks('post'): @@ -2258,7 +2258,7 @@ class LDAPRemoveReverseMember(LDAPModReverseMember): failed['member'][self.reverse_attr].append((attr, unicode(e))) # Update the member data. - entry_attrs = ldap.get_entry(dn, ['*']) + entry_attrs = ldap.get_entry(dn, attrs_list) self.obj.convert_attribute_members(entry_attrs, *keys, **options) for callback in self.get_callbacks('post'): |