maytasm commented on a change in pull request #12299:
URL: https://github.com/apache/druid/pull/12299#discussion_r819961340



##########
File path: docs/configuration/index.md
##########
@@ -651,7 +651,7 @@ You can configure Druid API error responses to hide 
internal information like th
 |--------|-----------|-------|
 |`druid.server.http.showDetailedJettyErrors`|When set to true, any error from 
the Jetty layer / Jetty filter includes the following fields  in the JSON 
response: `servlet`, `message`, `url`, `status`, and `cause`, if it exists. 
When set to false, the JSON response only includes `message`, `url`, and 
`status`. The field values remain unchanged.|true|
 |`druid.server.http.errorResponseTransform.strategy`|Error response transform 
strategy. The strategy controls how Druid transforms error responses from Druid 
services. When unset or set to `none`, Druid leaves error responses 
unchanged.|`none`|
-
+|`druid.server.http.returnJettyErrorsInJson`|When set to true, any error from 
the Jetty layer / Jetty filter is returned as a JSON response. If set to false, 
the error response is returned as HTML.|false|

Review comment:
       I think it will be more extensible and useful for this new config to 
take in a String to determine the type of response format. For example, 
"application/json", etc.




-- 
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]

Reply via email to