Skip to content

Commit cf59b1f

Browse files
srowendongjoon-hyun
authored andcommitted
[MINOR][SQL] Remove signature from Hive thriftserver exception
### 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 #43402 from srowen/HiveCookieSigner. Authored-by: Sean Owen <srowen@gmail.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
1 parent 4a0ed9c commit cf59b1f

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

+1-2
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)