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 <ayushsax...@apache.org>
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");

Reply via email to