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

(Updated July 17, 2019, 5:12 a.m.)


Review request for ranger, bhavik patel, Gautam Borad, Abhay Kulkarni, Madhan 
Neethiraj, Mehul Parikh, Nikhil P, Pradeep Agrawal, and Velmurugan Periasamy.


Bugs: RANGER-1644
    https://issues.apache.org/jira/browse/RANGER-1644


Repository: ranger


Description (updated)
-------

Allow individuals to use stronger encryption algorithms supported by JDK 1.8.
Applied mainly original patch from Endre 
Kovacs(https://reviews.apache.org/r/63209/).
Just created the new patch by fixing the test case which was failing on latest 
master branch.


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/util/PasswordUtils.java 
6480b17 
  
agents-common/src/test/java/org/apache/ranger/plugin/util/PasswordUtilsTest.java
 4e135aa 
  security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
bf50df3 
  
security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java
 2ddb5f3 
  security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml 
8ebeeb5 
  security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java 
dc845cf 


Diff: https://reviews.apache.org/r/71001/diff/1/


Testing (updated)
-------

Service Created And Updated : Service users password which was 
created(encrypted) with old algo i.e PBEWithMD5AndDES was updated successfully 
with new set of java 1.8 encryption algo's i.e 
https://docs.oracle.com/javase/8/docs/technotes/guides/security/SunProviders.html#SunJCEProvider
 .

-Service CRUD on fresh installation.
-Tested with different algorithm on jdk1.8
-Old Service CRUD after upgrade from 2.6 with jdk1.7 to 3.1maint with jdk1.8

PasswordUtilsTest: added new unit test and updated previous ones.
Added service update test: on service update new service password will be 
encrypted with the new algorithm.


Thanks,

Mateen Mansoori

Reply via email to