[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910336 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java --- @@ -75,6 +80,29 @@ public ClusterInfo getClusterInfoJSON() {

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910033 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -115,4 +124,59 @@ private void setOptHelper()

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910797 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -115,4 +124,59 @@ private void setOptHelper()

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143902334 --- Diff: common/src/main/java/org/apache/drill/common/util/DrillStringUtils.java --- @@ -1,203 +1,258 @@ -/** - * Licensed to the Apache

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910090 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -115,4 +124,59 @@ private void setOptHelper()

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910606 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -115,4 +124,59 @@ private void setOptHelper()

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/983#discussion_r143910712 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java --- @@ -115,4 +124,59 @@ private void setOptHelper()

[GitHub] drill pull request #983: MD-2769: DRILL-5819: Default value of security.admi...

2017-10-10 Thread bitblender
GitHub user bitblender opened a pull request: https://github.com/apache/drill/pull/983 MD-2769: DRILL-5819: Default value of security.admin.user_groups and security.admin.users is true The values for admin user/groups in the config file was incorrectly set to "true".