ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Endre Zoltan Kovacs via Review Board <nore...@reviews.apache.org>
Subject Re: Review Request 63209: RANGER-1644 changed crypto algorithm to a strong one
Date Fri, 03 Nov 2017 14:24:09 GMT

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

(Updated Nov. 3, 2017, 2:24 p.m.)


Review request for ranger.


Changes
-------

added test cases testing passwords with comas at a variety of places
fixed bug related to password containing one or more comas


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


Repository: ranger


Description
-------

changing outdate hash&crypto algorigthms: MD5&DES => SHA512&AES128


Diffs (updated)
-----

  agents-common/src/main/java/org/apache/ranger/plugin/util/PasswordUtils.java 58cdd3531 
  agents-common/src/test/java/org/apache/ranger/plugin/util/PasswordUtilsTest.java 4e135aaa7

  security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java da650747d 
  security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java 3dd761a2b

  security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml 9dfc03df1 
  security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java 976fd0cb8 


Diff: https://reviews.apache.org/r/63209/diff/3/

Changes: https://reviews.apache.org/r/63209/diff/2-3/


Testing
-------

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,

Endre Zoltan Kovacs


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