Merged HEAD-BUG-FIX (4.3/Cloud) to HEAD (4.3/Cloud)

57508: Merged V4.2-BUG-FIX (4.2.1) to HEAD-BUG-FIX (Cloud/4.3)
      57359: Merged V4.1-BUG-FIX (4.1.7) to V4.2-BUG-FIX (4.2.1)
         57305: Fix for MNT-9779


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@61838 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Alan Davis
2014-02-11 21:00:08 +00:00
parent bfc32bbcc9
commit 4632be0702
2 changed files with 87 additions and 49 deletions

View File

@@ -279,6 +279,10 @@ public abstract class BaseKerberosAuthenticationFilter extends BaseSSOAuthentica
// Check if the user is already authenticated
SessionUser user = getSessionUser(context, req, resp, true);
HttpSession httpSess = req.getSession(true);
if (user == null)
{
user = (SessionUser) httpSess.getAttribute("_alfAuthTicket");
}
// If the user has been validated and we do not require re-authentication then continue to
// the next filter
@@ -298,7 +302,7 @@ public abstract class BaseKerberosAuthenticationFilter extends BaseSSOAuthentica
}
// Check if the login page is being accessed, do not intercept the login page
if (hasLoginPage() && req.getRequestURI().endsWith(getLoginPage()))
if (checkLoginPage(req, resp))
{
if (getLogger().isDebugEnabled())
getLogger().debug("Login page requested, chaining ...");
@@ -461,6 +465,11 @@ public abstract class BaseKerberosAuthenticationFilter extends BaseSSOAuthentica
return false;
}
protected boolean checkLoginPage(HttpServletRequest req, HttpServletResponse resp)
{
return (hasLoginPage() && req.getRequestURI().endsWith(getLoginPage()));
}
/**
* JAAS callback handler
*

View File

@@ -19,6 +19,7 @@
package org.alfresco.repo.webdav.auth;
import java.io.IOException;
import java.io.PrintWriter;
import javax.servlet.ServletContext;
import javax.servlet.http.HttpServletRequest;
@@ -37,7 +38,6 @@ import org.apache.commons.logging.LogFactory;
public class KerberosAuthenticationFilter extends BaseKerberosAuthenticationFilter
{
// Debug logging
private static Log logger = LogFactory.getLog(KerberosAuthenticationFilter.class);
/* (non-Javadoc)
@@ -53,10 +53,39 @@ public class KerberosAuthenticationFilter extends BaseKerberosAuthenticationFilt
}
/* (non-Javadoc)
*
* @see org.alfresco.repo.webdav.auth.BaseSSOAuthenticationFilter#getLogger()
*/
@Override
protected Log getLogger() {
protected Log getLogger()
{
return logger;
}
@Override
protected boolean checkLoginPage(HttpServletRequest req, HttpServletResponse resp)
{
return (req.getRequestURI().endsWith("/jsp/login.jsp"));
}
/**
* Writes link to login page and refresh tag which cause user
* to be redirected to the login page.
*
* @param context ServletContext
* @param resp HttpServletResponse
* @param httpSess HttpSession
* @throws IOException
*/
protected void writeLoginPageLink(ServletContext context, HttpServletRequest req, HttpServletResponse resp) throws IOException
{
resp.setContentType(MIME_HTML_TEXT);
final PrintWriter out = resp.getWriter();
out.println("<html><head>");
out.println("<meta http-equiv=\"Refresh\" content=\"0; url=" + req.getContextPath() + "/faces/jsp/login.jsp?_alfRedirect=%2Falfresco%2Fwebdav\">");
out.println("</head><body><p>Please <a href=\"" + req.getContextPath() + "/faces/jsp/login.jsp?_alfRedirect=%2Falfresco%2Fwebdav\">log in</a>.</p>");
out.println("</body></html>");
out.close();
}
}