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

(Updated Jan. 8, 2014, 1:03 p.m.)


Review request for cloudstack and Abhinandan Prateek.


Changes
-------

this patch is for 4.3 branch


Bugs: CLOUDSTACK-5790 and CLOUDSTACK-5791
    https://issues.apache.org/jira/browse/CLOUDSTACK-5790
    https://issues.apache.org/jira/browse/CLOUDSTACK-5791


Repository: cloudstack-git


Description
-------

1. Reverted the commit for encrypting ldap hostname and port
2. decrypted ldap hostname and port during upgrade as they are not encrypted 
now.


Diffs (updated)
-----

  engine/schema/src/com/cloud/upgrade/dao/Upgrade421to430.java 471307a 
  
plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LDAPConfigCmd.java
 53d3877 
  
plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/response/LdapConfigurationResponse.java
 caabbe7 
  
plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapConfigurationVO.java
 2fb6332 
  
plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManagerImpl.java
 c2158f4 
  setup/db/db/schema-421to430.sql 284fb17 
  setup/db/db/schema-421to430.sql 7098632 

Diff: https://reviews.apache.org/r/16736/diff/


Testing
-------

manually


Thanks,

Rajani Karuturi

Reply via email to