[ 
https://issues.apache.org/jira/browse/MESOS-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14174492#comment-14174492
 ] 

Adam B commented on MESOS-1878:
-------------------------------

commit d10ac8e0b9561c2c08d4ccf9753c7471268513ee
Author: Cody Maloney <[email protected]>
Date:   Thu Oct 16 16:47:52 2014 -0700

    Escaped JSON object keys.
    
    All strings in JSON need to be escaped, including the keys in
    dictionaries / objects.
    
    Review: https://reviews.apache.org/r/26723

> Access to sandbox on slave from master UI does not show the sandbox contents
> ----------------------------------------------------------------------------
>
>                 Key: MESOS-1878
>                 URL: https://issues.apache.org/jira/browse/MESOS-1878
>             Project: Mesos
>          Issue Type: Bug
>          Components: webui
>    Affects Versions: 0.21.0
>            Reporter: Anindya Sinha
>            Assignee: Cody Maloney
>            Priority: Minor
>
> From master UI, clicking Sandbox to go to slave sandbox does not list the 
> sandbox contents. The directory path of the sandbox shows up fine, but not 
> the actual contents of the sandbox that is displayed below.
> Looks like the issue is it fails in the following GET from the corresponding 
> slave:
> http://<slave-host>:4891/files/browse.json?jsonp=angular.callbacks._9&path=<sandbox-path>
> Looking at the commits, I could confirm that the issue is not seen with 
> commit 'babb1c06ecf3077f292a19cfcbf1f1a4ed0e07b1'. Rolling back to a mesos 
> build with this commit being the last commit on mesos slave does not show 
> this behavior.
> Update: The issue has been introduced by the following 2 commits:
> ca2e8ef MESOS-1857 Fixed path::join() on older libstdc++ which lack back().
> b08fccf Switched path::join() to be variadic
> Note that the commit ca2e8ef fixes a build issue (on older libstd++) on top 
> of the commit b08fccf.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to