GitHub user aosagie opened a pull request:

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

    [SPARK-21176][WEB UI] Format worker page links to work with proxy

    ## What changes were proposed in this pull request?
    
    Several links on the worker page do not work correctly with the proxy 
because:
    1) They don't acknowledge the proxy
    2) They use relative paths (unlike the Application Page which uses full 
paths)
    
    This patch fixes that. It also fixes a mistake in the proxy's Location 
header parsing which caused it to incorrectly handle redirects.
    
    ## How was this patch tested?
    
    I checked the validity of every link with the proxy on and off.


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

    $ git pull https://github.com/aosagie/spark fix/proxy-links

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

    https://github.com/apache/spark/pull/18915.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 #18915
    
----
commit 2ab211b3c4d15c9f3fa8cab6af1f1d944bae3721
Author: Anderson Osagie <osa...@gmail.com>
Date:   2017-08-10T03:08:42Z

    [SPARK-21176][WEB UI] Format worker page links to work with proxy. Fix 
proxy location header creation

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to