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:

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

Review request for ranger.


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

Repository: ranger


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/


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


Endre Zoltan Kovacs

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