This is an automated email from the ASF dual-hosted git repository.
ayushsaxena pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git
The following commit(s) were added to refs/heads/master by this push:
new 7638cb1a3b0 Refactor Some Code. (#4887). (Ayush Saxena, reviewed by
Denys Kuzmenko)
7638cb1a3b0 is described below
commit 7638cb1a3b07713cc490aa2909a37037f89e08b4
Author: Ayush Saxena <[email protected]>
AuthorDate: Thu Nov 23 11:50:50 2023 +0530
Refactor Some Code. (#4887). (Ayush Saxena, reviewed by Denys Kuzmenko)
---
service/src/java/org/apache/hive/service/CookieSigner.java | 4 +---
service/src/test/org/apache/hive/service/TestCookieSigner.java | 2 +-
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/service/src/java/org/apache/hive/service/CookieSigner.java
b/service/src/java/org/apache/hive/service/CookieSigner.java
index 49ef2c606a7..867adc8fa26 100644
--- a/service/src/java/org/apache/hive/service/CookieSigner.java
+++ b/service/src/java/org/apache/hive/service/CookieSigner.java
@@ -76,10 +76,8 @@ public class CookieSigner {
String rawValue = signedStr.substring(0, index);
String currentSignature = getSignature(rawValue);
- LOG.debug("Signature generated for {} inside verify is {}", rawValue,
currentSignature);
if (!MessageDigest.isEqual(originalSignature.getBytes(),
currentSignature.getBytes())) {
- throw new IllegalArgumentException("Invalid sign, original = " +
originalSignature +
- " current = " + currentSignature);
+ throw new IllegalArgumentException("Invalid sign= " + originalSignature);
}
return rawValue;
}
diff --git a/service/src/test/org/apache/hive/service/TestCookieSigner.java
b/service/src/test/org/apache/hive/service/TestCookieSigner.java
index 54801d44400..a5a6454194d 100644
--- a/service/src/test/org/apache/hive/service/TestCookieSigner.java
+++ b/service/src/test/org/apache/hive/service/TestCookieSigner.java
@@ -70,7 +70,7 @@ public class TestCookieSigner {
try {
cs.verifyAndExtract(modifedSignedStr);
} catch (IllegalArgumentException e) {
- assertTrue(e.getMessage().startsWith("Invalid sign, original = "));
+ assertTrue(e.getMessage().startsWith("Invalid sign= "));
return;
}
fail("Expected IllegalArgumentException checking signature");