CLOUDSTACK-6210: LDAP:listLdapUsers api throws exception when we click on "Add 
LDAP Account" This occurs when ldap basedn is not configured. Throwing an IAE 
and a proper message is returned from the api call

Signed-off-by: Ian Duffy <i...@ianduffy.ie>
(cherry picked from commit 4552ec632201e7432afae7770f5854aaa244267c)
Signed-off-by: Rohit Yadav <rohit.ya...@shapeblue.com>

Conflicts:
        
plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/e3a12887
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/e3a12887
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/e3a12887

Branch: refs/heads/4.4
Commit: e3a1288763c3e91f49fcbd85c5e978e69bdbb4d1
Parents: 2de9388
Author: Rajani Karuturi <rajanikarut...@gmail.com>
Authored: Fri Mar 7 11:13:35 2014 +0530
Committer: Rohit Yadav <rohit.ya...@shapeblue.com>
Committed: Thu Dec 11 20:48:07 2014 +0530

----------------------------------------------------------------------
 .../ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e3a12887/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
----------------------------------------------------------------------
diff --git 
a/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
 
b/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
index afdf975..32dc404 100644
--- 
a/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
+++ 
b/plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
@@ -31,6 +31,8 @@ import javax.naming.directory.SearchResult;
 
 import org.apache.commons.lang.StringUtils;
 
+import org.apache.commons.lang.StringUtils;
+
 public class LdapUserManager {
 
     @Inject
@@ -193,4 +195,4 @@ public class LdapUserManager {
         }
         return context.search(basedn, generateSearchFilter(username), 
controls);
     }
-}
\ No newline at end of file
+}

Reply via email to