[native] Suppress connector values from being logged

The log contains the connector key and value pairs read from the
config files. The problem is that certain key-value pairs likely contain
secrets that should not be logged as this leaks credentials.

We cannot filter the keys to be able to print non-secret information
because there is no consistency for naming schemes of these properties
and we could miss newly ones added.

Author: czentgr

presto-native-execution/presto_cpp/main/PrestoServer.cpp

@@ -109,11 +109,14 @@ void enableChecksum() {
       });
 }
 
-std::string stringifyConnectorConfig(
+// Log only the catalog keys that are configured to avoid leaking
+// secret information. Some values represent secrets used to access
+// storage backends.
+std::string logConnectorConfigPropertyKeys(
     const std::unordered_map<std::string, std::string>& configs) {
   std::stringstream out;
   for (auto const& [key, value] : configs) {
-    out << "  " << key << "=" << value << "\n";
+    out << "  " << key << "\n";
   }
   return out.str();
 }
@@ -1220,8 +1223,8 @@ std::vector<std::string> PrestoServer::registerConnectors(
 
       auto connectorConf = util::readConfig(entry.path());
       PRESTO_STARTUP_LOG(INFO)
-          << "Registered properties from " << entry.path() << ":\n"
-          << stringifyConnectorConfig(connectorConf);
+          << "Registered catalog property keys from " << entry.path() << ":\n"
+          << logConnectorConfigPropertyKeys(connectorConf);
 
       std::shared_ptr<const velox::config::ConfigBase> properties =
           std::make_shared<const velox::config::ConfigBase>(