Author: ate
Date: Wed Jan 12 16:13:47 2011
New Revision: 1058217

URL: http://svn.apache.org/viewvc?rev=1058217&view=rev
Log:
JS2-1231: minor improvements to prevent unneeded recreation of session after 
login if the session was new already

Modified:
    
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/session/
   (props changed)
    
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/login/filter/PortalFilter.java
    
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/security/impl/shibboleth/ShibbolethPortalFilter.java

Propchange: 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/session/
------------------------------------------------------------------------------
--- svn:ignore (original)
+++ svn:ignore Wed Jan 12 16:13:47 2011
@@ -1,2 +1,2 @@
-target
+target
 surefire*.properties

Modified: 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/login/filter/PortalFilter.java
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/login/filter/PortalFilter.java?rev=1058217&r1=1058216&r2=1058217&view=diff
==============================================================================
--- 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/login/filter/PortalFilter.java
 (original)
+++ 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/login/filter/PortalFilter.java
 Wed Jan 12 16:13:47 2011
@@ -91,7 +91,7 @@ public class PortalFilter implements Fil
                     audit.logUserActivity(username, request.getRemoteAddr(), 
AuditActivity.AUTHENTICATION_SUCCESS, "PortalFilter");
                     PortalAuthenticationConfiguration 
authenticationConfiguration = (PortalAuthenticationConfiguration)
                         
cm.getComponent("org.apache.jetspeed.administration.PortalAuthenticationConfiguration");
-                    if 
(authenticationConfiguration.isCreateNewSessionOnLogin())
+                    if 
(authenticationConfiguration.isCreateNewSessionOnLogin() && httpSession != null 
&& !httpSession.isNew())
                     {
                         request.getSession().invalidate();
                     }

Modified: 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/security/impl/shibboleth/ShibbolethPortalFilter.java
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/security/impl/shibboleth/ShibbolethPortalFilter.java?rev=1058217&r1=1058216&r2=1058217&view=diff
==============================================================================
--- 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/security/impl/shibboleth/ShibbolethPortalFilter.java
 (original)
+++ 
portals/jetspeed-2/portal/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/security/impl/shibboleth/ShibbolethPortalFilter.java
 Wed Jan 12 16:13:47 2011
@@ -130,7 +130,7 @@ public class ShibbolethPortalFilter impl
                                        audit.logUserActivity(username, 
request.getRemoteAddr(), AuditActivity.AUTHENTICATION_SUCCESS, 
"ShibbolethFilter");
                                        PortalAuthenticationConfiguration 
authenticationConfiguration = (PortalAuthenticationConfiguration)
                                                        
cm.getComponent("org.apache.jetspeed.administration.PortalAuthenticationConfiguration");
-                                       if 
(authenticationConfiguration.isCreateNewSessionOnLogin())
+                                       if 
(authenticationConfiguration.isCreateNewSessionOnLogin() && httpSession != null 
&& !httpSession.isNew())
                                        {
                                                
request.getSession().invalidate();
                                        }



---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscr...@portals.apache.org
For additional commands, e-mail: jetspeed-dev-h...@portals.apache.org

Reply via email to