iain-buclaw-sociomantic opened a new pull request #399:
URL: https://github.com/apache/knox/pull/399


   ## What changes were proposed in this pull request?
   
   The Kill Application button is broken, KNOX matches and rewrites the URL 
twice into an invalid URL.
   
   This change just copies what is done for `/proxy/` urls.
   ```
   <filter name="YARNUI/yarn/outbound/apps">
       <content type="*/html">
           <apply path="(https?://[^/':,]+:[\d]+)?/proxy/[^']*" 
rule="YARNUI/yarn/outbound/apps/history"/>
           <!-- snip -->
       </content>
   </filter>
   <filter name="YARNUI/yarn/outbound/apps1">
       <content type="*/html">
           <apply path="/proxy/[^']*" 
rule="YARNUI/yarn/outbound/apps/history1"/>
           <!-- snip -->
       </content>
   </filter>
   ```
   It seems to work, though I'm not sure if it silently breaks other URLs just 
yet.
   
   ## How was this patch tested?
   
   Tested in local production cluster. Spawned up a new spark job and killed it 
via the Kill Application button in YARN.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to