redirect to the error page when IdP redirects to the authentication and the user...
authorlajoie <lajoie@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Wed, 11 May 2011 22:16:07 +0000 (22:16 +0000)
committerlajoie <lajoie@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Wed, 11 May 2011 22:16:07 +0000 (22:16 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/branches/REL_2@3028 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

doc/RELEASE-NOTES.txt
src/main/java/edu/internet2/middleware/shibboleth/idp/authn/AuthenticationEngine.java

index 2ee903f..fadd58d 100644 (file)
@@ -18,6 +18,7 @@ Changes in Release 2.3.0
 [SIDP-480] - Update POM to add plugin versions, use / publish to Shib.net Repo, and attach generated source and Javadocs
 [SIDP-482] - JSP pages should HTML-encode any strings they handle
 [SIDP-483] - Log Completed, Unencrypted SAML Assertion
+[SIDP-484] - Login stops at AuthnEngine with an empty page
 [SIDP-485] - <img> inside HTML pages must have alt attribute.[SIDP-486] - login.jsp page contains old wiki link
 [SIDP-487] - More login.jsp changes (CR/LF issues, missing period)
 [SIDP-488] - PeerEntityId property not set on SAML queries
index ec161a9..e137f57 100644 (file)
@@ -206,8 +206,8 @@ public class AuthenticationEngine extends HttpServlet {
 
         LoginContext loginContext = HttpServletHelper.getLoginContext(storageService, getServletContext(), httpRequest);
         if (loginContext == null) {
-            LOG.error("Incoming request does not have attached login context");
-            throw new ServletException("Incoming request does not have attached login context");
+            LOG.warn("No login context available, unable to proceed with authentication");
+            forwardRequest("/error.jsp", httpRequest, httpResponse);
         }
 
         if (!loginContext.getAuthenticationAttempted()) {