Mailing-List: contact issues-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Date: Fri, 19 Jun 2015 03:56:00 +0000 (UTC)
From: "Lefty Leverenz (JIRA)" <jira@apache.org>
To: issues@hive.apache.org
Message-ID: <JIRA.12718967.1402096307000.116942.1434686160487@Atlassian.JIRA>
In-Reply-To: <JIRA.12718967.1402096307000@Atlassian.JIRA>
References: <JIRA.12718967.1402096307000@Atlassian.JIRA>
 <JIRA.12718967.1402096307402@arcas>
Subject: [jira] [Commented] (HIVE-7193) Hive should support additional LDAP
 authentication parameters
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HIVE-7193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14592969#comment-14592969 ] 

Lefty Leverenz commented on HIVE-7193:
--------------------------------------

Doc review (parameter descriptions):

*hive.server2.authentication.ldap.groupDNPattern*
{code}
+        "COLON-separated list of patterns to use to find DNs for group entities in this directory \n" +
+        "use %s where the actual group name is to be substituted for.\n" +
+        "For example: CN=%s,CN=Groups,DC=subdomain,DC=domain,DC=com."),
{code}
Please add a period at end of first line and start second line with initial capital "Use ...."  Also, why is the example a comma-separated list when the description says colon-separated?

*hive.server2.authentication.ldap.groupFilter*
{code}
+        "COMMA-separated list of LDAP Group names (short name not full DNs) \n" +
+        " For example: HiveAdmins,HadoopAdmins,Administrators"),
{code}
Again, end the first line with a period.  Remove the space at beginning of second line.

*hive.server2.authentication.ldap.userDNPattern* 
{code}
+        "COLON-separated list of patterns to use to find DNs for users in this directory \n" +
+        "use %s where the actual group name is to be substituted for.\n" +
+        "For example: CN=%s,CN=Users,DC=subdomain,DC=domain,DC=com." +
+        "COLON-seperated list of Base DNs for User entities in the LDAP directory"),
{code}
Again, add period to first line and start second line "Use".  Why is the list comma-separated?  Does the fourth line belong somewhere else?  (It misspells "separated" too.)

*hive.server2.authentication.ldap.userFilter*
{code}
+        "COMMA-separated list of LDAP usernames (just short names, not full DNs) \n" +
+        "For example: hiveuser,impalauser,hiveadmin,hadoopadmin"),
{code}
Add period at end of first line.

*hive.server2.authentication.ldap.customLDAPQuery*
{code}
+        "A full LDAP query that LDAP Atn provider uses to execute against LDAP Server \n" +
+        "If this query return a null resultset, the LDAP Provider fails the Authentication request \n" +
+        ", succeeds otherwise." +
+        "For example: (&(objectClass=group)(objectClass=top)(instanceType=4)(cn=Domain*)) \n" +
+        "(&(objectClass=person)(|(sAMAccountName=admin)(|(memberOf=CN=Domain Admins,CN=Users,DC=domain,DC=com)" +
+        "(memberOf=CN=Administrators,CN=Builtin,DC=domain,DC=com))))"),
{code}
Add a period at end of first line.  Second line:  "If this query returns ..." (add the s to return) and move comma from start of third line to end of second line (or move "request" to third line).   

> Hive should support additional LDAP authentication parameters
> -------------------------------------------------------------
>
>                 Key: HIVE-7193
>                 URL: https://issues.apache.org/jira/browse/HIVE-7193
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 0.10.0
>            Reporter: Mala Chikka Kempanna
>            Assignee: Naveen Gangam
>         Attachments: HIVE-7193.2.patch, HIVE-7193.3.patch, HIVE-7193.4.patch, HIVE-7193.patch, LDAPAuthentication_Design_Doc.docx, LDAPAuthentication_Design_Doc_V2.docx
>
>
> Currently hive has only following authenticator parameters for LDAP authentication for hiveserver2:
> {code:xml}
> <property> 
>   <name>hive.server2.authentication</name> 
>   <value>LDAP</value> 
> </property> 
> <property> 
>   <name>hive.server2.authentication.ldap.url</name> 
>   <value>ldap://our_ldap_address</value> 
> </property> 
> {code}
> We need to include other LDAP properties as part of hive-LDAP authentication like below:
> {noformat}
> a group search base -> dc=domain,dc=com 
> a group search filter -> member={0} 
> a user search base -> dc=domain,dc=com 
> a user search filter -> sAMAAccountName={0} 
> a list of valid user groups -> group1,group2,group3 
> {noformat}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)