From 3e5ddc82d1237d63e572bccc9cf28789749e1e57 Mon Sep 17 00:00:00 2001 From: HHoflittlefish777 <77738092+HHoflittlefish777@users.noreply.github.com> Date: Fri, 17 Nov 2023 14:42:51 +0800 Subject: [PATCH] [improvement](log) log desensitization without displaying user info (#26912) (#27167) --- .../src/main/java/org/apache/doris/httpv2/rest/LoadAction.java | 3 ++- .../java/org/apache/doris/httpv2/rest/RestBaseController.java | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java index 71afe476ee865c..9bfbbccfa8a6b0 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java +++ b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java @@ -323,7 +323,8 @@ private Object executeWithClusterToken(HttpServletRequest request, String db, if (!Strings.isNullOrEmpty(request.getQueryString())) { redirectUrl += request.getQueryString(); } - LOG.info("Redirect url: {}", redirectUrl); + LOG.info("Redirect url: {}", "http://" + redirectAddr.getHostname() + ":" + + redirectAddr.getPort() + urlObj.getPath()); RedirectView redirectView = new RedirectView(redirectUrl); redirectView.setContentType("text/html;charset=utf-8"); redirectView.setStatusCode(org.springframework.http.HttpStatus.TEMPORARY_REDIRECT); diff --git a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/RestBaseController.java b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/RestBaseController.java index 7b4b4dd54985c1..ac388349889ee8 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/RestBaseController.java +++ b/fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/RestBaseController.java @@ -105,7 +105,8 @@ public RedirectView redirectTo(HttpServletRequest request, TNetworkAddress addr) if (!Strings.isNullOrEmpty(request.getQueryString())) { redirectUrl += request.getQueryString(); } - LOG.info("redirect url: {}", redirectUrl); + LOG.info("Redirect url: {}", "http://" + addr.getHostname() + ":" + + addr.getPort() + urlObj.getPath()); RedirectView redirectView = new RedirectView(redirectUrl); redirectView.setContentType("text/html;charset=utf-8"); redirectView.setStatusCode(org.springframework.http.HttpStatus.TEMPORARY_REDIRECT);