From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Tue, 19 Oct 2010 20:27:39 +0000 (+0000)
Subject: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49991
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=68d7bb80e57da2acf1ae18eedba4dc30649de255;p=tomcat7.0

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49991
Ensure servlet request listeners are fired for the login and error pages during FORM authentication.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1024397 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/authenticator/FormAuthenticator.java b/java/org/apache/catalina/authenticator/FormAuthenticator.java
index 8e14b225f..f8ed35ce2 100644
--- a/java/org/apache/catalina/authenticator/FormAuthenticator.java
+++ b/java/org/apache/catalina/authenticator/FormAuthenticator.java
@@ -327,7 +327,10 @@ public class FormAuthenticator
             context.getServletContext().getRequestDispatcher
             (config.getLoginPage());
         try {
-            disp.forward(request.getRequest(), response);
+            if (context.fireRequestInitEvent(request)) {
+                disp.forward(request.getRequest(), response);
+                context.fireRequestDestroyEvent(request);
+            }
         } catch (Throwable t) {
             ExceptionUtils.handleThrowable(t);
             String msg = sm.getString("formAuthenticator.forwardLoginFail");
@@ -357,7 +360,10 @@ public class FormAuthenticator
             context.getServletContext().getRequestDispatcher
             (config.getErrorPage());
         try {
-            disp.forward(request.getRequest(), response);
+            if (context.fireRequestInitEvent(request)) {
+                disp.forward(request.getRequest(), response);
+                context.fireRequestDestroyEvent(request);
+            }
         } catch (Throwable t) {
             ExceptionUtils.handleThrowable(t);
             String msg = sm.getString("formAuthenticator.forwardErrorFail");
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 4c5cf3019..175c0db6c 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -40,6 +40,10 @@
   <subsection name="Catalina">
     <changelog>
       <fix>
+        <bug>49991</bug>: Ensure servlet request listeners are fired for
+        the login and error pages during FORM authentication. (markt)
+      </fix>
+      <fix>
         <bug>50107</bug>: When removing a Host via JMX, do not attempt to
         destroy the host&apos;s pipeline twice. Patch provided by Eiji
         Takahashi. (markt)