You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2009/12/19 02:43:30 UTC

svn commit: r892415 - in /tomcat/trunk/java/org/apache/catalina: authenticator/AuthenticatorBase.java connector/Request.java

Author: kkolinko
Date: Sat Dec 19 01:43:29 2009
New Revision: 892415

URL: http://svn.apache.org/viewvc?rev=892415&view=rev
Log:
Followup to rev.889716
The new AuthenticatorBase property needs a setter.
Also, corrected a javadoc misprint.

Modified:
    tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java
    tomcat/trunk/java/org/apache/catalina/connector/Request.java

Modified: tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java?rev=892415&r1=892414&r2=892415&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java (original)
+++ tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java Sat Dec 19 01:43:29 2009
@@ -386,6 +386,31 @@
         this.securePagesWithPragma = securePagesWithPragma;
     }    
 
+    /**
+     * Return the flag that states if we should change the session ID of an
+     * existing session upon successful authentication.
+     * 
+     * @return <code>true</code> to change session ID upon successful
+     *         authentication, <code>false</code> to do not perform the change.
+     */
+    public boolean getChangeSessionIdOnAuthentication() {
+        return changeSessionIdOnAuthentication;
+    }
+
+    /**
+     * Set the value of the flag that states if we should change the session ID
+     * of an existing session upon successful authentication.
+     * 
+     * @param changeSessionIdOnAuthentication
+     *            <code>true</code> to change session ID upon successful
+     *            authentication, <code>false</code> to do not perform the
+     *            change.
+     */
+    public void setChangeSessionIdOnAuthentication(
+            boolean changeSessionIdOnAuthentication) {
+        this.changeSessionIdOnAuthentication = changeSessionIdOnAuthentication;
+    }
+
     // --------------------------------------------------------- Public Methods
 
 

Modified: tomcat/trunk/java/org/apache/catalina/connector/Request.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Request.java?rev=892415&r1=892414&r2=892415&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/connector/Request.java (original)
+++ tomcat/trunk/java/org/apache/catalina/connector/Request.java Sat Dec 19 01:43:29 2009
@@ -2273,7 +2273,7 @@
 
     /**
      * Change the ID of the session that this request is associated with. There
-     * are several things that may trigger an ID change. These include mmoving
+     * are several things that may trigger an ID change. These include moving
      * between nodes in a cluster and session fixation prevention during the
      * authentication process.
      * 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r892415 - in /tomcat/trunk/java/org/apache/catalina: authenticator/AuthenticatorBase.java connector/Request.java

Posted by Mark Thomas <ma...@apache.org>.
On 19/12/2009 01:43, kkolinko@apache.org wrote:
> Author: kkolinko
> Date: Sat Dec 19 01:43:29 2009
> New Revision: 892415
> 
> URL: http://svn.apache.org/viewvc?rev=892415&view=rev
> Log:
> Followup to rev.889716
> The new AuthenticatorBase property needs a setter.

Thanks. Good catch.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org