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

    https://github.com/apache/nifi/pull/3185#discussion_r237256616
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/java/org/apache/nifi/web/filter/LogoutFilter.java
 ---
    @@ -50,7 +50,7 @@ public void doFilter(ServletRequest request, 
ServletResponse response, FilterCha
                 final ServletContext apiContext = 
servletContext.getContext("/nifi-api");
                 
apiContext.getRequestDispatcher("/access/knox/logout").forward(request, 
response);
             } else {
    -            ((HttpServletResponse) response).sendRedirect("../login");
    +            ((HttpServletResponse) response).sendRedirect("../nifi/login");
    --- End diff --
    
    Can you test, `sendRedirect("login");`? I think the reason it breaks is 
because we are already in the `nifi` directory, but the `..` makes us leave it.


---

Reply via email to