You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by bb...@apache.org on 2020/06/17 20:33:13 UTC
[nifi] branch master updated: NIFI-7537 - Small fix to make the
LDAP connection timeout property a String rather than a Long. Fixes cast
error in LDAP libs.
This is an automated email from the ASF dual-hosted git repository.
bbende pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nifi.git
The following commit(s) were added to refs/heads/master by this push:
new 2b25ccc NIFI-7537 - Small fix to make the LDAP connection timeout property a String rather than a Long. Fixes cast error in LDAP libs.
2b25ccc is described below
commit 2b25ccc2988693f116bc7d10e98de6271bbbe1a6
Author: Nathan Gough <th...@gmail.com>
AuthorDate: Tue Jun 16 14:04:21 2020 -0400
NIFI-7537 - Small fix to make the LDAP connection timeout property a String rather than a Long. Fixes cast error in LDAP libs.
---
.../src/main/java/org/apache/nifi/ldap/LdapProvider.java | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/nifi-nar-bundles/nifi-ldap-iaa-providers-bundle/nifi-ldap-iaa-providers/src/main/java/org/apache/nifi/ldap/LdapProvider.java b/nifi-nar-bundles/nifi-ldap-iaa-providers-bundle/nifi-ldap-iaa-providers/src/main/java/org/apache/nifi/ldap/LdapProvider.java
index 8ae30a0..4570faf 100644
--- a/nifi-nar-bundles/nifi-ldap-iaa-providers-bundle/nifi-ldap-iaa-providers/src/main/java/org/apache/nifi/ldap/LdapProvider.java
+++ b/nifi-nar-bundles/nifi-ldap-iaa-providers-bundle/nifi-ldap-iaa-providers/src/main/java/org/apache/nifi/ldap/LdapProvider.java
@@ -16,11 +16,6 @@
*/
package org.apache.nifi.ldap;
-import java.util.HashMap;
-import java.util.Map;
-import java.util.concurrent.TimeUnit;
-import javax.naming.Context;
-import javax.net.ssl.SSLContext;
import org.apache.commons.lang3.StringUtils;
import org.apache.nifi.authentication.AuthenticationResponse;
import org.apache.nifi.authentication.LoginCredentials;
@@ -55,6 +50,12 @@ import org.springframework.security.ldap.search.FilterBasedLdapUserSearch;
import org.springframework.security.ldap.search.LdapUserSearch;
import org.springframework.security.ldap.userdetails.LdapUserDetails;
+import javax.naming.Context;
+import javax.net.ssl.SSLContext;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.concurrent.TimeUnit;
+
/**
* Abstract LDAP based implementation of a login identity provider.
*/
@@ -237,7 +238,7 @@ public class LdapProvider implements LoginIdentityProvider {
if (StringUtils.isNotBlank(rawTimeout)) {
try {
final long timeout = (long) FormatUtils.getPreciseTimeDuration(rawTimeout, TimeUnit.MILLISECONDS);
- baseEnvironment.put(environmentKey, timeout);
+ baseEnvironment.put(environmentKey, String.valueOf(timeout));
} catch (final IllegalArgumentException iae) {
throw new ProviderCreationException(String.format("The %s '%s' is not a valid time duration", configurationProperty, rawTimeout));
}