diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/VitroHttpServlet.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/VitroHttpServlet.java index a8e4e349c..d4ff77d19 100644 --- a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/VitroHttpServlet.java +++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/VitroHttpServlet.java @@ -102,7 +102,9 @@ public class VitroHttpServlet extends HttpServlet { protected boolean isAuthorizedToDisplayPage(HttpServletRequest request, HttpServletResponse response, Actions actions) { // Record restricted pages so we won't return to them on logout - LogoutRedirector.recordRestrictedPageUri(request); + if (!actions.isEmpty()) { + LogoutRedirector.recordRestrictedPageUri(request); + } if (PolicyHelper.isAuthorizedForActions(request, actions)) { log.debug("Servlet '" + this.getClass().getSimpleName() diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java index 515e2224f..8257d7094 100644 --- a/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java +++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/FreemarkerHttpServlet.java @@ -86,15 +86,12 @@ public class FreemarkerHttpServlet extends VitroHttpServlet { Configuration config = getConfig(vreq); vreq.setAttribute("freemarkerConfig", config); - ResponseValues responseValues; - - // This method does a redirect if the required authorizations are not met, so just return. - if (!isAuthorizedToDisplayPage(request, response, requiredActions(vreq))) { - return; - } else { - responseValues = processRequest(vreq); - } + // This method does a redirect if the required authorizations are not met, so just return. + if (!isAuthorizedToDisplayPage(request, response, requiredActions(vreq))) { + return; + } + ResponseValues responseValues = processRequest(vreq); doResponse(vreq, response, responseValues); } catch (TemplateProcessingException e) {