You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/07/07 20:05:20 UTC
[GitHub] [druid] gianm opened a new pull request, #12755: Retain CSP configuration in ServerConfig constructor.
gianm opened a new pull request, #12755:
URL: https://github.com/apache/druid/pull/12755
Without this change, CliIndexer would not apply custom CSP headers and would revert to the default. Other server types respect it just fine, because they use an instance created by JsonConfigProvider, which sets properties directly. Fortunately, the default is a secure choice. Feature originally added in #12609.
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] abhishekagarwal87 merged pull request #12755: Retain CSP configuration in ServerConfig constructor.
Posted by GitBox <gi...@apache.org>.
abhishekagarwal87 merged PR #12755:
URL: https://github.com/apache/druid/pull/12755
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] abhishekagarwal87 commented on pull request #12755: Retain CSP configuration in ServerConfig constructor.
Posted by GitBox <gi...@apache.org>.
abhishekagarwal87 commented on PR #12755:
URL: https://github.com/apache/druid/pull/12755#issuecomment-1179011863
Merged since failure was unrelated.
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org