portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a..@apache.org
Subject svn commit: r548358 - /portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java
Date Mon, 18 Jun 2007 13:54:32 GMT
Author: ate
Date: Mon Jun 18 06:54:32 2007
New Revision: 548358

URL: http://svn.apache.org/viewvc?view=rev&rev=548358
Log:
Fix for: JS2-735
See: https://issues.apache.org/jira/browse/JS2-735

Note: this a to resolve a Jetty 6 issue (probably a bug) which cannot handle session.invalidate()
called from a HttpSessionBindingListener.unboundValue(...).

I'm removing the extra, extra safeguard forcing invalidation when the PortletApplicationSessionMonitor
is removed from the session itself.
This isn't absolutely needed, but it now allows an application session to "detach" itself
from the Portal Session synchronization by removing the Monitor itself (which would be a weird
action anyway). 

Modified:
    portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java

Modified: portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java
URL: http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java?view=diff&rev=548358&r1=548357&r2=548358
==============================================================================
--- portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java
(original)
+++ portals/jetspeed-2/trunk/components/portal/src/java/org/apache/jetspeed/container/session/PortalSessionsManagerImpl.java
Mon Jun 18 06:54:32 2007
@@ -119,9 +119,6 @@
             {
                 ((PortletApplicationSessionMonitor)iter.next()).invalidateSession();
             }
-            // To make sure its gone.
-            // You better not remove the psm from the portal session yourself ;)
-            psm.invalidateSession();
         }
     }
 
@@ -231,9 +228,6 @@
         if ( psr != null )
         {
             psr.sessionMonitors.remove(pasm.getContextPath());
-            // To make sure its gone.
-            // You better not remove the pasm from the session yourself ;)
-            pasm.invalidateSession();
         }
     }
 



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


Mime
View raw message