GitHub user mgaido91 opened a pull request:

    https://github.com/apache/spark/pull/21523

    [SPARK-24506][UI] Add UI filters also to thriftserver tab

    ## What changes were proposed in this pull request?
    
    Currently, `spark.ui.filters` are not applied to the pages added in the 
thriftserver tab. This happens because the tab is attached after the UI is 
started. This can allow unauthorized access to the pages.
    
    The PR adds the filters also to the pages which are added by the 
thriftserver tab.
    
    ## How was this patch tested?
    
    manual tests (without the patch, starting the thriftserver with `--conf 
spark.ui.filters=org.apache.hadoop.security.authentication.server.AuthenticationFilter
 --conf 
spark.org.apache.hadoop.security.authentication.server.AuthenticationFilter.params="type=simple"`
 you can access `http://localhost:4040/sqlserver`; with the patch, 401 is the 
response as for the other pages).


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mgaido91/spark SPARK-24506

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/21523.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 #21523
    
----
commit 086a4f62b95b455e9e789ea820b0e1f8b7e8d643
Author: Marco Gaido <marcogaido91@...>
Date:   2018-06-10T10:44:01Z

    [SPARK-24506][UI] Add UI filters also to thriftserver tab

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to