[
https://issues.apache.org/jira/browse/DRILL-7626?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17059420#comment-17059420
]
ASF GitHub Bot commented on DRILL-7626:
---------------------------------------
paul-rogers commented on issue #2025: DRILL-7626: Add ability to set HTTP
response headers
URL: https://github.com/apache/drill/pull/2025#issuecomment-599107346
Very cool feature, thanks!
I wonder, is it possible to add a unit test for this? In working with the
plugin registry, I notice we have few (or no) tests of the UI, and this caused
lots of wasted time from reviewers. If we have tests that act as a client to
Drill, can we add a quick test for this feature? If not, can we add something
so that it is easier to ask others to add unit tests? I think @dobesv, may have
added something recently.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> Add ability to set HTTP response headers
> ----------------------------------------
>
> Key: DRILL-7626
> URL: https://issues.apache.org/jira/browse/DRILL-7626
> Project: Apache Drill
> Issue Type: Sub-task
> Reporter: Igor Guzenko
> Assignee: Igor Guzenko
> Priority: Major
> Labels: ready-to-commit
> Fix For: 1.18.0
>
>
> It should be possible to make response headers configurable under common
> options path
> *drill.exec.http.jetty.server.response.headers*, like
> {code:none}
> jetty: {
> server: {
> response: {
> # any response headers with constant values may be configured like
> this
> headers: {
> "Custom Header Name": "Text Value",
> "BooleanHeader": true
> }
> }
> }
> }
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
