Repository: hadoop Updated Branches: refs/heads/branch-2 ee4ace3ed -> 25dcb8863 refs/heads/branch-2.8 379c2f4c0 -> ddd82d24b refs/heads/trunk 23f937e3b -> 70c756d35
HADOOP-12795. KMS does not log detailed stack trace for unexpected errors. Contributed by Chris Nauroth. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/70c756d3 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/70c756d3 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/70c756d3 Branch: refs/heads/trunk Commit: 70c756d35e6ed5608ce82d1a6fbfb02e19af5ecf Parents: 23f937e Author: cnauroth <cnaur...@apache.org> Authored: Thu Feb 11 16:52:58 2016 -0800 Committer: cnauroth <cnaur...@apache.org> Committed: Thu Feb 11 16:52:58 2016 -0800 ---------------------------------------------------------------------- hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++ .../hadoop/crypto/key/kms/server/KMSExceptionsProvider.java | 2 ++ 2 files changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/70c756d3/hadoop-common-project/hadoop-common/CHANGES.txt ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 634baed..0a19800 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -1695,6 +1695,9 @@ Release 2.8.0 - UNRELEASED "hadoop.workaround.non.threadsafe.getpwuid" is different between core-default.xml and NativeIO.java (Akira AJISAKA via cmccabe) + HADOOP-12795. KMS does not log detailed stack trace for unexpected errors. + (cnauroth) + Release 2.7.3 - UNRELEASED INCOMPATIBLE CHANGES http://git-wip-us.apache.org/repos/asf/hadoop/blob/70c756d3/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java b/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java index 5cb0885..aa9d299 100644 --- a/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java +++ b/hadoop-common-project/hadoop-kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSExceptionsProvider.java @@ -86,12 +86,14 @@ public class KMSExceptionsProvider implements ExceptionMapper<Exception> { status = Response.Status.FORBIDDEN; } else if (exception instanceof IOException) { status = Response.Status.INTERNAL_SERVER_ERROR; + log(status, throwable); } else if (exception instanceof UnsupportedOperationException) { status = Response.Status.BAD_REQUEST; } else if (exception instanceof IllegalArgumentException) { status = Response.Status.BAD_REQUEST; } else { status = Response.Status.INTERNAL_SERVER_ERROR; + log(status, throwable); } if (doAudit) { KMSWebApp.getKMSAudit().error(KMSMDCFilter.getUgi(),
