pranavbhole commented on code in PR #17045:
URL: https://github.com/apache/druid/pull/17045#discussion_r1758157435
##########
docs/configuration/index.md:
##########
@@ -616,9 +616,10 @@ the [HDFS input
source](../ingestion/input-sources.md#hdfs-input-source).
You can set the following property to specify permissible protocols for
the [HTTP input source](../ingestion/input-sources.md#http-input-source).
-|Property|Possible values|Description|Default|
-|--------|---------------|-----------|-------|
-|`druid.ingestion.http.allowedProtocols`|List of protocols|Allowed protocols
for the HTTP input source.|`["http", "https"]`|
+|Property| Possible values | Description
|Default|
+|--------|-----------------|-------------------------------------------------------------------------------------------------------------------------------------------|-------|
+|`druid.ingestion.http.allowedProtocols`| List of protocols | Allowed
protocols for the HTTP input source.
|`["http", "https"]`|
+|`druid.ingestion.http.allowedHeaders`| A list of permitted request headers
for the HTTP input source. By default, the list is empty, which means all
headers are allowed in the ingestion specification. |`[]`|
Review Comment:
if we we want to restrict headers then druid.ingestion.http.allowedHeaders
should be non empty.
##########
docs/configuration/index.md:
##########
@@ -616,9 +616,10 @@ the [HDFS input
source](../ingestion/input-sources.md#hdfs-input-source).
You can set the following property to specify permissible protocols for
the [HTTP input source](../ingestion/input-sources.md#http-input-source).
-|Property|Possible values|Description|Default|
-|--------|---------------|-----------|-------|
-|`druid.ingestion.http.allowedProtocols`|List of protocols|Allowed protocols
for the HTTP input source.|`["http", "https"]`|
+|Property| Possible values | Description
|Default|
+|--------|-----------------|-------------------------------------------------------------------------------------------------------------------------------------------|-------|
+|`druid.ingestion.http.allowedProtocols`| List of protocols | Allowed
protocols for the HTTP input source.
|`["http", "https"]`|
+|`druid.ingestion.http.allowedHeaders`| A list of permitted request headers
for the HTTP input source. By default, the list is empty, which means all
headers are allowed in the ingestion specification. |`[]`|
Review Comment:
if we want to restrict headers then druid.ingestion.http.allowedHeaders
should be non empty.
--
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]
