Refactor Some Code. (#4887). (Ayush Saxena, reviewed by Denys Kuzmenko)

apache · Nov 23, 2023 · 7638cb1 · 7638cb1
1 parent ae89034
commit 7638cb1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git 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 String verifyAndExtract(String signedStr) {
     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
@@ -70,7 +70,7 @@ public void testVerifyAndExtractInvalidSignature() {
     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");