Skip to content

Commit 1302775

Browse files
wangyumsrowen
authored andcommitted
[MINOR][SQL] Remove signature from Hive thriftserver exception (apache#676)
### What changes were proposed in this pull request? Don't return expected signature to caller in Hive thriftserver exception ### Why are the changes needed? Please see private discussion ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Existing tests ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#43402 from srowen/HiveCookieSigner. Authored-by: Sean Owen <srowen@gmail.com> (cherry picked from commit cf59b1f) Signed-off-by: Dongjoon Hyun <dhyun@apple.com> Co-authored-by: Sean Owen <srowen@gmail.com>
1 parent 1545db0 commit 1302775

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

sql/hive-thriftserver/src/main/java/org/apache/hive/service/CookieSigner.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,7 @@ public String verifyAndExtract(String signedStr) {
8181
LOG.debug("Signature generated for " + rawValue + " inside verify is " + currentSignature);
8282
}
8383
if (!MessageDigest.isEqual(originalSignature.getBytes(), currentSignature.getBytes())) {
84-
throw new IllegalArgumentException("Invalid sign, original = " + originalSignature +
85-
" current = " + currentSignature);
84+
throw new IllegalArgumentException("Invalid sign");
8685
}
8786
return rawValue;
8887
}

0 commit comments

Comments
 (0)