[improvement](log) log desensitization without displaying user info (#…

…26912) (#27167)
apache · Nov 17, 2023 · 3e5ddc8 · 3e5ddc8
1 parent 0043f16
commit 3e5ddc8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 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
@@ -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
@@ -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);