ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tom Beerbower" <tbeerbo...@hortonworks.com>
Subject Re: Review Request 31159: Views: Login redirection fails in Ambari when trying to access view
Date Wed, 18 Feb 2015 20:22:15 GMT


> On Feb. 18, 2015, 7:59 p.m., Nate Cole wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilter.java,
line 68
> > <https://reviews.apache.org/r/31159/diff/1/?file=867868#file867868line68>
> >
> >     This could be configurable.  The instance looks like it's loaded via an injector,
so access to Configuration.class should be there.

It could be but right now there is no requirement and Ambari Web needs to see it in this format.

Thanks.


- Tom


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


On Feb. 18, 2015, 8:17 p.m., Tom Beerbower wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31159/
> -----------------------------------------------------------
> 
> (Updated Feb. 18, 2015, 8:17 p.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley, Nate Cole, and Yusaku Sako.
> 
> 
> Bugs: AMBARI-9694
>     https://issues.apache.org/jira/browse/AMBARI-9694
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Access a view (i.e. "http://c6501.ambari.apache.org:8080/views/TEZ/0.5.2.2.2.2.0-869/tez/#/tez-app/application_1423719241211_0050")
from a browser where I have already logged in to Ambari, the link works fine.
> When I access it from another browser, I get a 403 with no redirect to the login page.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilter.java
bedffbb 
>   ambari-server/src/test/java/org/apache/ambari/server/security/authorization/AmbariAuthorizationFilterTest.java
f3c6400 
> 
> Diff: https://reviews.apache.org/r/31159/diff/
> 
> 
> Testing
> -------
> 
> Manual tested.
> New unit test added.
> 
> All existing tests pass ...
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message