[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218575627 ## File path: server/src/main/java/org/apache/druid/server/StatusResource.java ## @@ -47,6 +50,7 @@ @Path("/status")

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217071931 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217072220 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r216886161 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,32

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218575985 ## File path: server/src/main/java/org/apache/druid/server/security/TLSUtils.java ## @@ -39,28 +44,77 @@ public static

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r218576728 ## File path: docs/content/operations/tls-support.md ## @@ -31,7 +31,19 @@ values for the below mentioned configs among

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-09-18 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r217071194 ## File path: server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java ## @@ -256,6 +259,32

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-08-17 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r210964758 ## File path: docs/content/operations/tls-support.md ## @@ -31,6 +31,16 @@ values for the below mentioned configs among

[GitHub] gianm commented on a change in pull request #6076: Mutual TLS support

2018-08-17 Thread GitBox
gianm commented on a change in pull request #6076: Mutual TLS support URL: https://github.com/apache/incubator-druid/pull/6076#discussion_r210964888 ## File path: docs/content/operations/tls-support.md ## @@ -31,6 +31,16 @@ values for the below mentioned configs among