Martin Peřina has uploaded a new change for review. Change subject: aaa: Fix the name of property for list of ldap servers ......................................................................
aaa: Fix the name of property for list of ldap servers Fixes the name property for list of ldap servers to be the same as in ConfigValue -> "LdapServers" (of course in aaa part with prefix "config." Change-Id: I796385144fb163d2ddf0298bcb653cf88bc77772 Bug-Url: https://bugzilla.redhat.com/1111071 Signed-off-by: Martin Perina <[email protected]> --- M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/DirectorySearcher.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java 2 files changed, 3 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/68/28968/1 diff --git a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/DirectorySearcher.java b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/DirectorySearcher.java index a9f86f4..2554211 100644 --- a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/DirectorySearcher.java +++ b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/DirectorySearcher.java @@ -57,7 +57,7 @@ public List<?> find(final LdapQueryData queryData, final long resultCount) { final String domainName = queryData.getDomain(); List<String> ldapServerURIs = - Arrays.asList(configuration.getProperty("config.LDAPServers").split(",")); + Arrays.asList(configuration.getProperty("config.LdapServers").split(",")); List<String> editableLdapServerURIs = new ArrayList<>(ldapServerURIs); if (log.isDebugEnabled()) { log.debug("Ldap server list: " + StringUtils.join(ldapServerURIs, ", ")); diff --git a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java index 32d134d..b119032 100644 --- a/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java +++ b/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java @@ -34,7 +34,7 @@ throw new RuntimeException(e); } } - if (StringUtils.isBlank(conf.getProperty("config.LDAPServers"))) { + if (StringUtils.isBlank(conf.getProperty("config.LdapServers"))) { try { LdapSRVLocator locator = new LdapSRVLocator(); @@ -48,7 +48,7 @@ String ldapURI = locator.constructURI("ldap", address, "389").toString(); ldapServers.add(ldapURI); } - conf.setProperty("config.LDAPServers", StringUtils.join(ldapServers, ",")); + conf.setProperty("config.LdapServers", StringUtils.join(ldapServers, ",")); } } catch (Exception ex) { throw new RuntimeException(ex); -- To view, visit http://gerrit.ovirt.org/28968 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I796385144fb163d2ddf0298bcb653cf88bc77772 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
