Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ambari.apache.org
Content-Type: multipart/alternative;
 boundary="===============7587266388363199876=="
MIME-Version: 1.0
Subject: Re: Review Request 32804: Falcon View to use doAs impersonation for
 user.name in Ambari Views
From: "Alejandro Fernandez" <afernandez@hortonworks.com>
To: "Venkat Ranganathan" <n.r.v@live.com>,
 "Yusaku Sako" <yusaku@hortonworks.com>,
 "Sumit Mohanty" <smohanty@hortonworks.com>,
 "Tom Beerbower" <tbeerbower@hortonworks.com>
Cc: "Alejandro Fernandez" <afernandez@hortonworks.com>,
 "Ambari" <dev@ambari.apache.org>
Date: Fri, 03 Apr 2015 22:54:21 -0000
Message-ID: <20150403225421.16791.4199@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Alejandro Fernandez" <noreply@reviews.apache.org>
References: <20150403043124.16792.10100@reviews.apache.org>
In-Reply-To: <20150403043124.16792.10100@reviews.apache.org>
Reply-To: "Alejandro Fernandez" <afernandez@hortonworks.com>

--===============7587266388363199876==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


> On April 3, 2015, 4:31 a.m., Venkat Ranganathan wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/view/HttpImpersonatorImpl.java, line 267
> > <https://reviews.apache.org/r/32804/diff/1/?file=914294#file914294line267>
> >
> >     Not sure if this can be made more generic.   If there can be some generic header that can be explicitly specified for this purpose and we will check for that to set user name instead of looking for Falcon headers?

Technically, we shouldn't have logic to detect Falcon View. Instead, the Falcon View can pass an ImpersonatorSetting object whose doAsParamName is "user.name" instead of the default "doAs".


- Alejandro


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


On April 3, 2015, 12:46 a.m., Alejandro Fernandez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32804/
> -----------------------------------------------------------
> 
> (Updated April 3, 2015, 12:46 a.m.)
> 
> 
> Review request for Ambari, Sumit Mohanty, Tom Beerbower, Venkat Ranganathan, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-10341
>     https://issues.apache.org/jira/browse/AMBARI-10341
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Falcon View needs to use the HttpImpersonator interface in Ambari Views so it can proxy using the header user.name
> 
> HttpImpersonatorImpl.java should only allow this if the request headers indicate it is for the Falcon View.
> 
> 
> Diffs
> -----
> 
>   ambari-server/pom.xml 896a202 
>   ambari-server/src/main/java/org/apache/ambari/server/view/HttpImpersonatorImpl.java fe4e180 
>   ambari-server/src/test/java/org/apache/ambari/server/view/HttpImpersonatorImplTest.java 935b488 
>   ambari-views/pom.xml 8fcb5ae 
>   ambari-views/src/main/java/org/apache/ambari/view/HttpImpersonator.java 02ce101 
>   ambari-views/src/main/java/org/apache/ambari/view/ImpersonatorResponse.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/32804/diff/
> 
> 
> Testing
> -------
> 
> Ran local unit tests in HttpImpersonatorImplTest.java passed.
> 
> 
> Thanks,
> 
> Alejandro Fernandez
> 
>


--===============7587266388363199876==--