commit c198a8e6dcfa1f25ac8bb7b404b3e71ebc74a577
author Gustaf Lundh <gustaflh@axis.com> Fri Oct 13 12:48:42 2017 +0200
committer Gustaf Lundh <gustaflh@axis.com> Fri Oct 13 12:50:13 2017 +0200
tree 52f95c20120fe0769f5b82053d18cb9efb99f0b6
parent aedd872a981680e11171a00f4304c3cd90fb4e73

Fix issue where static pages where not served

Static pages like /static/ or /Documentation/ should not need login.

This fixes issue where the header logo is not visible on login page
and documentation link is not visible in polygerrit.

Change-Id: Id670e19dcc15e1b2c8e940f410e7291351f9129f

src/main/java/com/googlesource/gerrit/plugins/loginredirect/LoginRedirectFilter.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/loginredirect/LoginRedirectFilter.java b/src/main/java/com/googlesource/gerrit/plugins/loginredirect/LoginRedirectFilter.java
index 9dd7e54..ec70e99 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/loginredirect/LoginRedirectFilter.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/loginredirect/LoginRedirectFilter.java
@@ -40,7 +40,11 @@
     if (!httpReq.getContextPath().isEmpty()) {
       path = path.substring(httpReq.getContextPath().length());
     }
-    if (path.startsWith("/login") || path.startsWith("/a/") || sessionProvider.get().isSignedIn()) {
+    if (path.startsWith("/login") ||
+        path.startsWith("/a/") ||
+        path.startsWith("/Documentation/") ||
+        path.startsWith("/static/") ||
+        sessionProvider.get().isSignedIn()) {
       chain.doFilter(request, response);
     } else {
       ((HttpServletResponse) response).sendRedirect(getLoginRedirectUrl(httpReq));