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".
These options have no meaningful static default values. At runtime, the
process user/groups should be used as the default admin user/groups. New
accessors have been added for these options and these have to be used instead
of the usual option manager accessors. Dummy default strings are used in the
config file, which when returned by the usual options manager, indicate that
the user has not changed these options.
This change implements the above-mentioned items and also adds a UI section
to see the current admin user/groups. This UI section is only shown to
authenticated admin users.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bitblender/drill DRILL-5819
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/983.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #983
----
commit 3ba5401a3f6780dbbffd36ef2a9f67b12089a9ef
Author: karthik <[email protected]>
Date: 2017-10-04T18:23:04Z
MD-2769: DRILL-5819: Default value of security.admin.user_groups and
security.admin.users is true
----
---
