You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Oleksiy Sayankin <ol...@gmail.com> on 2018/02/16 14:55:56 UTC

Re: Review Request 65483: HIVE-18541 Secure HS2 web UI with PAM

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65483/
-----------------------------------------------------------

(Updated Feb. 16, 2018, 4:55 p.m.)


Review request for hive, Lefty Leverenz, Rui Li, Lenni Kuff, Szehon Ho, and Szehon Ho.


Repository: hive-git


Description
-------

Secure HS2 web UI with PAM. Add two new properties

    hive.server2.webui.use.pam
    Default value: false
    Description: If true, the HiveServer2 WebUI will be secured with PAM

    hive.server2.webui.pam.authenticator
    Default value: org.apache.hive.http.security.PamAuthenticator
    Description: Class for PAM authentication


Diffs
-----

  common/pom.xml aaeecc0 
  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java adb9b9b 
  common/src/java/org/apache/hive/http/HttpServer.java 2a8f7ae 
  common/src/java/org/apache/hive/http/security/PamAuthenticator.java PRE-CREATION 
  common/src/java/org/apache/hive/http/security/PamConstraint.java PRE-CREATION 
  common/src/java/org/apache/hive/http/security/PamConstraintMapping.java PRE-CREATION 
  common/src/java/org/apache/hive/http/security/PamLoginService.java PRE-CREATION 
  common/src/java/org/apache/hive/http/security/PamUserIdentity.java PRE-CREATION 
  service/src/java/org/apache/hive/service/server/HiveServer2.java 2e57b60 
  service/src/test/org/apache/hive/service/server/TestHS2HttpServerPam.java PRE-CREATION 
  service/src/test/org/apache/hive/service/server/TestHS2HttpServerPamConfiguration.java PRE-CREATION 


Diff: https://reviews.apache.org/r/65483/diff/8/


Testing
-------


Thanks,

Oleksiy Sayankin

Re: Review Request 65483: HIVE-18541 Secure HS2 web UI with PAM

Posted by Oleksiy Sayankin <ol...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65483/#review199314
-----------------------------------------------------------


Ship it!




Ship It!

- Oleksiy Sayankin


On Feb. 16, 2018, 4:55 p.m., Oleksiy Sayankin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65483/
> -----------------------------------------------------------
> 
> (Updated Feb. 16, 2018, 4:55 p.m.)
> 
> 
> Review request for hive, Lefty Leverenz, Rui Li, Lenni Kuff, Szehon Ho, and Szehon Ho.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Secure HS2 web UI with PAM. Add two new properties
> 
>     hive.server2.webui.use.pam
>     Default value: false
>     Description: If true, the HiveServer2 WebUI will be secured with PAM
> 
>     hive.server2.webui.pam.authenticator
>     Default value: org.apache.hive.http.security.PamAuthenticator
>     Description: Class for PAM authentication
> 
> 
> Diffs
> -----
> 
>   common/pom.xml aaeecc0 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java adb9b9b 
>   common/src/java/org/apache/hive/http/HttpServer.java 2a8f7ae 
>   common/src/java/org/apache/hive/http/security/PamAuthenticator.java PRE-CREATION 
>   common/src/java/org/apache/hive/http/security/PamConstraint.java PRE-CREATION 
>   common/src/java/org/apache/hive/http/security/PamConstraintMapping.java PRE-CREATION 
>   common/src/java/org/apache/hive/http/security/PamLoginService.java PRE-CREATION 
>   common/src/java/org/apache/hive/http/security/PamUserIdentity.java PRE-CREATION 
>   service/src/java/org/apache/hive/service/server/HiveServer2.java 2e57b60 
>   service/src/test/org/apache/hive/service/server/TestHS2HttpServerPam.java PRE-CREATION 
>   service/src/test/org/apache/hive/service/server/TestHS2HttpServerPamConfiguration.java PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/65483/diff/8/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Oleksiy Sayankin
> 
>