[Bug] BE download_files function exists log print sensitive msg apach…

…e#28592
LuGuangming · Dec 25, 2023 · 45c1c4f · 45c1c4f
1 parent 4495f81
commit 45c1c4f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/be/src/olap/task/engine_clone_task.cpp b/be/src/olap/task/engine_clone_task.cpp
@@ -31,13 +31,13 @@
 #include <memory>
 #include <mutex>
 #include <ostream>
+#include <regex>
 #include <set>
 #include <shared_mutex>
 #include <system_error>
 #include <unordered_map>
 #include <unordered_set>
 #include <utility>
-#include <regex>
 
 #include "common/config.h"
 #include "common/logging.h"
@@ -547,7 +547,8 @@ Status EngineCloneTask::_download_files(DataDir* data_dir, const std::string& re
             }
             if (local_file_size != file_size) {
                 LOG(WARNING) << "download file length error"
-                             << ", remote_path=" << _mask_token(remote_file_url) << ", file_size=" << file_size
+                             << ", remote_path=" << _mask_token(remote_file_url)
+                             << ", file_size=" << file_size
                              << ", local_file_size=" << local_file_size;
                 return Status::InternalError("downloaded file size is not equal");
             }

diff --git a/be/src/olap/task/engine_clone_task.h b/be/src/olap/task/engine_clone_task.h
@@ -85,7 +85,7 @@ class EngineCloneTask : public EngineTask {
                           bool* allow_incremental_clone);
 
     Status _release_snapshot(const std::string& ip, int port, const std::string& snapshot_path);
-    
+
     std::string _mask_token(const std::string& str);
 
 private: