cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Abhinandan Prateek" <aprat...@apache.org>
Subject Re: Review Request 12469: WIP: New implementation of LDAP authentication
Date Fri, 12 Jul 2013 11:02:33 GMT


> On July 12, 2013, 9:50 a.m., Abhinandan Prateek wrote:
> > Ship It!
> 
> Sebastien Goasguen wrote:
>     Abhi, I had already applied it to the ldapplugin branch. Did you apply the patch
somewhere else ?

No thats fine. I realised that after I have marked "ship it". Maybe Ian can close this request
now.


- Abhinandan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12469/#review23072
-----------------------------------------------------------


On July 11, 2013, 1:32 p.m., Ian Duffy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12469/
> -----------------------------------------------------------
> 
> (Updated July 11, 2013, 1:32 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Sebastien Goasguen.
> 
> 
> Bugs: CLOUDSTACK-2014
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Adds a new implementation of the LDAP authenticator. 
> This is a work in progress and has been placed here to show my progress within the GSoC
program.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/configuration/ConfigurationService.java 381fcad 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 83999b6 
>   api/src/org/apache/cloudstack/api/ResponseGenerator.java d8d07cb 
>   api/src/org/apache/cloudstack/api/command/admin/ldap/LDAPConfigCmd.java 38f58ec 
>   api/src/org/apache/cloudstack/api/command/admin/ldap/LDAPRemoveCmd.java 5159fba 
>   api/src/org/apache/cloudstack/api/response/LDAPConfigResponse.java bbeec63 
>   api/src/org/apache/cloudstack/api/response/LDAPRemoveResponse.java 0feec5b 
>   client/tomcatconf/applicationContext.xml.in 610fdfd 
>   client/tomcatconf/commands.properties.in f4b41a7 
>   client/tomcatconf/componentContext.xml.in 1fbec61 
>   client/tomcatconf/nonossComponentContext.xml.in ffa6281 
>   client/tomcatconf/simulatorComponentContext.xml.in 7225e84 
>   plugins/api/rate-limit/src/org/apache/cloudstack/ratelimit/ApiRateLimitServiceImpl.java
7d1b43a 
>   plugins/user-authenticators/ldap/pom.xml 5c45f11 
>   plugins/user-authenticators/ldap/src/com/cloud/server/auth/LDAPUserAuthenticator.java
d928a5b 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapAddConfigurationCmd.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapDeleteConfigurationCmd.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapListAllUsersCmd.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapListConfigurationCmd.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapUserSearchCmd.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/response/LdapConfigurationResponse.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/response/LdapUserResponse.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapConfiguration.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapConfigurationVO.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapContextFactory.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManager.java PRE-CREATION

>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManagerImpl.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUser.java PRE-CREATION

>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUserManager.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapUtils.java PRE-CREATION

>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/NoLdapUserMatchingQueryException.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/NoSuchLdapUserException.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/dao/LdapConfigurationDao.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/dao/LdapConfigurationDaoImpl.java
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/BasicNamingEnumerationImpl.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapAddConfigurationCmdSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapAuthenticatorSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapConfigurationDaoImplSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapConfigurationResponseSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapConfigurationSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapConfigurationVO.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapContextFactorySpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapDeleteConfigurationCmdSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapListAllUsersCmdSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapListConfigurationCmdSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapManagerImplSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapSearchUserCmdSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapUserManagerSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapUserResponseSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapUserSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapUtilsSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/NoLdapUserMatchingQueryExceptionSpec.groovy
PRE-CREATION 
>   plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/NoSuchLdapUserExceptionSpec.groovy
PRE-CREATION 
>   server/src/com/cloud/api/ApiResponseHelper.java 090d9c2 
>   server/src/com/cloud/configuration/Config.java d3ed718 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 8d8b126 
>   server/src/com/cloud/server/ManagementServerImpl.java a0b9daa 
>   server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 7a61978 
>   setup/db/db/schema-410to420.sql 1e9c201 
>   tools/apidoc/gen_toc.py 33a7e75 
>   ui/scripts/globalSettings.js 1ae73b7 
> 
> Diff: https://reviews.apache.org/r/12469/diff/
> 
> 
> Testing
> -------
> 
> Unit tests have been wrote for the plugin and have 92% code coverage.
> The code successfully complies and I have done manual testing. That is, adding an LDAP
server(Windows AD based and OpenLDAP based) configuring the credentials and attributes for
it and testing a login.
> Updated integration tests will be submitted at stage in the future.
> 
> 
> Thanks,
> 
> Ian Duffy
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message