Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1203#discussion_r181254054
  
    --- Diff: exec/java-exec/src/main/resources/rest/index.ftl ---
    @@ -67,9 +73,15 @@
                     <td>${i}</td>
                     <td id="address" >${drillbit.getAddress()}<#if 
drillbit.isCurrent()>
                         <span class="label label-info" 
id="current">Current</span>
    +                    <#else>
    +                    <a 
onclick="popOutRemoteDbitUI('${drillbit.getAddress()}','${drillbit.getHttpPort()}');"
 style="cursor:pointer;color:blue" title="Open in new window"><span 
class="glyphicon glyphicon-new-window"/></a>
    --- End diff --
    
    This will only work if authentication is disabled. Shouldn't we have that 
check here to display ?


---

Reply via email to