Merge pull request #15576 from cdapio/story/stacktraceToLogger

[CDAP-20834] Replace usage of printStack with logger
cdapio · Apr 4, 2024 · 3aa4496 · 3aa4496
2 parents c6e7429 + fe8bbfe
commit 3aa4496
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/...cdap-data-pipeline-base/src/main/java/io/cdap/cdap/datapipeline/service/OAuthHandler.java b/...cdap-data-pipeline-base/src/main/java/io/cdap/cdap/datapipeline/service/OAuthHandler.java
@@ -51,11 +51,14 @@
 import javax.ws.rs.QueryParam;
 import javax.ws.rs.core.HttpHeaders;
 import javax.ws.rs.core.MediaType;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * OAuth handler.
  */
 public class OAuthHandler extends AbstractSystemHttpServiceHandler {
+  private static final Logger LOG = LoggerFactory.getLogger(OAuthHandler.class);
   private static final String API_VERSION = "v1";
   private static final Gson GSON = new GsonBuilder()
     .setPrettyPrinting()
@@ -362,7 +365,7 @@ private static class OAuthServiceException extends Exception {
 
     void respond(HttpServiceResponder responder) {
       if (status == HttpURLConnection.HTTP_INTERNAL_ERROR) {
-        printStackTrace();
+        LOG.error("An internal error has occurred", this);
         responder.sendError(status, "Internal error");
       } else {
         responder.sendError(status, getMessage());