zemin-piao commented on code in PR #12950:
URL: https://github.com/apache/druid/pull/12950#discussion_r956238549
##########
docs/configuration/index.md:
##########
@@ -787,6 +787,14 @@ All Druid components can communicate with each other over
HTTP.
|`druid.global.http.unusedConnectionTimeout`|The timeout for idle connections
in connection pool. The connection in the pool will be closed after this
timeout and a new one will be established. This timeout should be less than
`druid.global.http.readTimeout`. Set this timeout = ~90% of
`druid.global.http.readTimeout`|`PT4M`|
|`druid.global.http.numMaxThreads`|Maximum number of I/O worker
threads|`max(10, ((number of cores * 17) / 16 + 2) + 30)`|
+### Common endpoints Configuration
+
+This section contains the configuration options for endpoints that are
supported by all processes.
+
+|Property| Description
|Default|
+|--------|----------------------------------------------------------------------------------------------------------------------------------------------|-------|
+|`druid.server.hiddenProperties`| If property names or substring of property
names (case insensitive) is in this list, responses of the `/status/properties`
endpoint do not show these properties
|`["druid.s3.accessKey","druid.s3.secretKey","druid.metadata.storage.connector.password"]`|
Review Comment:
Sure thing, we can already include substrings already at
`druid.server.hiddenProperties `. I'd propose to add those substrings mentioned
at https://github.com/apache/druid/issues/12063: `["password", "key", "token",
"pwd"]`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
