Oved Ourfali has uploaded a new change for review. Change subject: aaa: Get users by domain returns partial list of users ......................................................................
aaa: Get users by domain returns partial list of users Only partial list of users is returned when querying domain users Change-Id: Ie4c121b0edd6966c518b46aa8e98c24be6cb74c8 Topic: AAA Bug-Url: https://bugzilla.redhat.com/1121410 Signed-off-by: Ravi Nori <rn...@redhat.com> (cherry picked from commit 117fb46e47cb4bdf5b5e3f2d5b5b4a0dc3c844f7) --- M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java 1 file changed, 6 insertions(+), 12 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/57/30557/1 diff --git a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java index 911396f..80404af 100644 --- a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java +++ b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthz.java @@ -81,7 +81,7 @@ public KerberosLdapAuthz() { } - public ExtMap queryGroups(ExtMap input, ExtMap output, long pageSize) { + public ExtMap queryGroups(ExtMap input, ExtMap output) { LdapQueryData queryData = new LdapQueryDataImpl(); queryData.setLdapQueryType(LdapQueryType.searchGroups); queryData.setDomain(getDirectoryName()); @@ -98,14 +98,12 @@ List<LdapGroup> ldapGroups = (List<LdapGroup>) ldapResult.getReturnValue(); List<ExtMap> results = new ArrayList<>(); for (LdapGroup ldapGroup : ldapGroups) { - if (results.size() < pageSize) { - results.add(mapLdapGroup(ldapGroup)); - } + results.add(mapLdapGroup(ldapGroup)); } return output.mput(Authz.InvokeKeys.QUERY_RESULT, results); } - public ExtMap queryUsers(ExtMap input, ExtMap output, long pageSize) { + public ExtMap queryUsers(ExtMap input, ExtMap output) { LdapQueryData queryData = new LdapQueryDataImpl(); queryData.setLdapQueryType(LdapQueryType.searchUsers); queryData.setDomain(getDirectoryName()); @@ -122,11 +120,8 @@ ); List<LdapUser> ldapUsers = (List<LdapUser>) ldapResult.getReturnValue(); List<ExtMap> results = new ArrayList<>(); - int addedUsers = 0; for (LdapUser ldapUser : ldapUsers) { - if (addedUsers++ < pageSize) { - results.add(mapLdapUser(ldapUser)); - } + results.add(mapLdapUser(ldapUser)); } return output.mput(Authz.InvokeKeys.QUERY_RESULT, results); } @@ -198,16 +193,15 @@ private void doQueryExecute(ExtMap input, ExtMap output) { Opaque opaque = input.<Opaque> get(Authz.InvokeKeys.QUERY_OPAQUE); - int pageSize = input.<Integer> get(Authz.InvokeKeys.PAGE_SIZE); if (opaque.getQueryInfo() == null) { output.mput(Authz.InvokeKeys.QUERY_RESULT, null); } else { if (opaque.getQueryInfo().<ExtUUID> get(Authz.InvokeKeys.QUERY_ENTITY).equals(Authz.QueryEntity.GROUP)) { - queryGroups(opaque.getQueryInfo(), output, pageSize); + queryGroups(opaque.getQueryInfo(), output); } else if (opaque.getQueryInfo() .<ExtUUID> get(Authz.InvokeKeys.QUERY_ENTITY) .equals(Authz.QueryEntity.PRINCIPAL)) { - queryUsers(opaque.getQueryInfo(), output, pageSize); + queryUsers(opaque.getQueryInfo(), output); } opaque.resetQueryInfo(); } -- To view, visit http://gerrit.ovirt.org/30557 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie4c121b0edd6966c518b46aa8e98c24be6cb74c8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Ravi Nori <rn...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
