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");