summaryrefslogtreecommitdiffstats
path: root/ipaserver/plugins
diff options
context:
space:
mode:
authorFlorence Blanc-Renaud <flo@redhat.com>2016-09-29 13:46:05 +0200
committerMartin Basti <mbasti@redhat.com>2016-11-17 01:01:05 +0100
commitefb3700389ff46244189fa95779484eb099d63b4 (patch)
treebaf94ed279dd53abe7a1799b908fffec05cb7e19 /ipaserver/plugins
parentce2bb47cca03eda1ff85f4725abb92c639f34ecc (diff)
downloadfreeipa-efb3700389ff46244189fa95779484eb099d63b4.tar.gz
freeipa-efb3700389ff46244189fa95779484eb099d63b4.tar.xz
freeipa-efb3700389ff46244189fa95779484eb099d63b4.zip
Fix ipa migrate-ds when it finds a search reference
When ipa migrate-ds finds user entries and a search reference, it complains that the LDAP search did not return any result and does not migrate the entries or the groups. The issue comes from LDAPClient._convert_result which returns an empty result list when the input is a search reference. In turn LDAPClient.find_entries assumes that the empty result list corresponds to a Search Result Done and returns without any entry. The fix examines first the objtype returned by self.conn.result3. If it is a search result done, then the loop can be exited. Otherwise (referral or entry), _convert_result is called and the result (if not empty) is appended to the list of returned entries. https://fedorahosted.org/freeipa/ticket/6358 Reviewed-By: Martin Basti <mbasti@redhat.com>
Diffstat (limited to 'ipaserver/plugins')
-rw-r--r--ipaserver/plugins/migration.py3
1 files changed, 1 insertions, 2 deletions
diff --git a/ipaserver/plugins/migration.py b/ipaserver/plugins/migration.py
index 5090a2646..72abd141f 100644
--- a/ipaserver/plugins/migration.py
+++ b/ipaserver/plugins/migration.py
@@ -746,8 +746,7 @@ migration process might be incomplete\n''')
entries, truncated = ds_ldap.find_entries(
search_filter, ['*'], search_bases[ldap_obj_name],
scope,
- time_limit=0, size_limit=-1,
- search_refs=True # migrated DS may contain search references
+ time_limit=0, size_limit=-1
)
except errors.NotFound:
if not options.get('continue',False):