You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ga...@apache.org on 2017/05/05 18:13:20 UTC
ranger git commit: RANGER-1560 : Code Improvement To Follow Best
Practices
Repository: ranger
Updated Branches:
refs/heads/ranger-0.7 07fc81040 -> ca69b58ca
RANGER-1560 : Code Improvement To Follow Best Practices
Signed-off-by: Gautam Borad <ga...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/ca69b58c
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/ca69b58c
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/ca69b58c
Branch: refs/heads/ranger-0.7
Commit: ca69b58ca9b634dcc90b49fe6f4a580df3c49213
Parents: 07fc810
Author: Ankita Sinha <an...@apache.org>
Authored: Fri May 5 16:58:26 2017 +0530
Committer: Gautam Borad <ga...@apache.org>
Committed: Fri May 5 23:43:07 2017 +0530
----------------------------------------------------------------------
.../ranger/plugin/model/RangerPolicyResourceSignature.java | 4 ++--
.../ranger/plugin/store/RangerServiceResourceSignature.java | 2 +-
.../ranger/plugin/model/TestRangerPolicyResourceSignature.java | 2 +-
.../apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ranger/blob/ca69b58c/agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicyResourceSignature.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicyResourceSignature.java b/agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicyResourceSignature.java
index c63b54d..dab63fa 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicyResourceSignature.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicyResourceSignature.java
@@ -45,7 +45,7 @@ public class RangerPolicyResourceSignature {
_policy = policy;
PolicySerializer serializer = new PolicySerializer(_policy);
_string = serializer.toString();
- _hash = DigestUtils.md5Hex(_string);
+ _hash = DigestUtils.sha256Hex(_string);
}
/**
@@ -59,7 +59,7 @@ public class RangerPolicyResourceSignature {
} else {
_string = string;
}
- _hash = DigestUtils.md5Hex(_string);
+ _hash = DigestUtils.sha256Hex(_string);
}
String asString() {
http://git-wip-us.apache.org/repos/asf/ranger/blob/ca69b58c/agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java b/agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java
index fca1c1d..48b19fa 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java
@@ -31,7 +31,7 @@ public class RangerServiceResourceSignature {
public RangerServiceResourceSignature(RangerServiceResource serviceResource) {
_string = ServiceResourceSerializer.toString(serviceResource);
- _hash = DigestUtils.md5Hex(_string);
+ _hash = DigestUtils.sha256Hex(_string);
}
String asString() {
http://git-wip-us.apache.org/repos/asf/ranger/blob/ca69b58c/agents-common/src/test/java/org/apache/ranger/plugin/model/TestRangerPolicyResourceSignature.java
----------------------------------------------------------------------
diff --git a/agents-common/src/test/java/org/apache/ranger/plugin/model/TestRangerPolicyResourceSignature.java b/agents-common/src/test/java/org/apache/ranger/plugin/model/TestRangerPolicyResourceSignature.java
index eae8aa5..410bc69 100644
--- a/agents-common/src/test/java/org/apache/ranger/plugin/model/TestRangerPolicyResourceSignature.java
+++ b/agents-common/src/test/java/org/apache/ranger/plugin/model/TestRangerPolicyResourceSignature.java
@@ -104,7 +104,7 @@ public class TestRangerPolicyResourceSignature {
// String rep of a null policy is an empty string! and its hash is sha of empty string!
RangerPolicyResourceSignature signature = new RangerPolicyResourceSignature((String)null);
Assert.assertEquals("", signature.asString());
- Assert.assertEquals(DigestUtils.md5Hex(""), signature.getSignature());
+ Assert.assertEquals(DigestUtils.sha256Hex(""), signature.getSignature());
}
/*
http://git-wip-us.apache.org/repos/asf/ranger/blob/ca69b58c/ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java
----------------------------------------------------------------------
diff --git a/ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java b/ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java
index 430212b..02ead75 100644
--- a/ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java
+++ b/ranger-hive-utils/src/main/java/org/apache/ranger/authorization/hive/udf/RangerUdfMaskHash.java
@@ -47,7 +47,7 @@ class MaskHashTransformer extends AbstractTransformer {
@Override
String transform(final String value) {
- return DigestUtils.md5Hex(value);
+ return DigestUtils.sha256Hex(value);
}
@Override