diff options
author | Juan Gonzalez | 2017-01-20 11:48:47 +0000 |
---|---|---|
committer | Juan Gonzalez | 2017-02-15 15:55:58 +0000 |
commit | d27f1bdb863b15aac929c81fc55fd536259ab9b2 (patch) | |
tree | ede910f814904511ade134a96424e8809ecdb33e /bundles/org.eclipse.equinox.http.servlet/src | |
parent | d071f6db6f8663fe86b9a372def46efc5b0bfeb8 (diff) | |
download | rt.equinox.bundles-d27f1bdb863b15aac929c81fc55fd536259ab9b2.tar.gz rt.equinox.bundles-d27f1bdb863b15aac929c81fc55fd536259ab9b2.tar.xz rt.equinox.bundles-d27f1bdb863b15aac929c81fc55fd536259ab9b2.zip |
Bug 510753 [http servlet] Invoke session listeners when unregistering them. Fix ListenerRegistration.equals so it gets actually removed from available EventListenersY20170216-1515Y20170216-1450Y20170216-1000I20170221-2000I20170220-2000I20170219-2000I20170218-2000I20170217-2000I20170216-2000I20170216-0140I20170215-2000
Signed-off-by: Juan Gonzalez <juangon@gmail.com>
Diffstat (limited to 'bundles/org.eclipse.equinox.http.servlet/src')
3 files changed, 55 insertions, 6 deletions
diff --git a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/context/ContextController.java b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/context/ContextController.java index a2d4a7227..019c6a3fb 100644 --- a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/context/ContextController.java +++ b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/context/ContextController.java @@ -15,9 +15,7 @@ import java.net.URI; import java.net.URISyntaxException; import java.security.AccessController; import java.util.*; -import java.util.concurrent.ConcurrentHashMap; -import java.util.concurrent.ConcurrentMap; -import java.util.concurrent.ConcurrentSkipListSet; +import java.util.concurrent.*; import java.util.concurrent.atomic.AtomicReference; import java.util.regex.Pattern; import javax.servlet.*; @@ -786,6 +784,12 @@ public class ContextController { } } + public Map<String, HttpSessionAdaptor> getActiveSessions() { + checkShutdown(); + + return activeSessions; + } + public Set<EndpointRegistration<?>> getEndpointRegistrations() { checkShutdown(); diff --git a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/registration/ListenerRegistration.java b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/registration/ListenerRegistration.java index ccbaf4d84..99c9e6625 100644 --- a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/registration/ListenerRegistration.java +++ b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/registration/ListenerRegistration.java @@ -12,11 +12,12 @@ package org.eclipse.equinox.http.servlet.internal.registration; import java.lang.reflect.*; -import java.util.EventListener; -import java.util.List; +import java.util.*; import javax.servlet.*; +import javax.servlet.http.*; import org.eclipse.equinox.http.servlet.internal.context.ContextController; import org.eclipse.equinox.http.servlet.internal.context.ContextController.ServiceHolder; +import org.eclipse.equinox.http.servlet.internal.servlet.HttpSessionAdaptor; import org.osgi.framework.wiring.BundleWiring; import org.osgi.service.http.runtime.dto.ListenerDTO; @@ -65,6 +66,18 @@ public class ListenerRegistration extends Registration<EventListener, ListenerDT super.destroy(); + if (classes.contains(HttpSessionBindingListener.class) || + classes.contains(HttpSessionAttributeListener.class) || + classes.contains(HttpSessionListener.class)) { + + Map<String, HttpSessionAdaptor> activeSessions = + contextController.getActiveSessions(); + + for (HttpSessionAdaptor adaptor : activeSessions.values()) { + adaptor.invokeSessionListeners(classes, super.getT()); + } + } + if (classes.contains(ServletContextListener.class)) { ServletContextListener servletContextListener = (ServletContextListener)super.getT(); @@ -88,7 +101,7 @@ public class ListenerRegistration extends Registration<EventListener, ListenerDT ListenerRegistration listenerRegistration = (ListenerRegistration)obj; - return super.getT().equals(listenerRegistration.getT()); + return listenerRegistration.getT().equals(super.getT()); } @Override diff --git a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/servlet/HttpSessionAdaptor.java b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/servlet/HttpSessionAdaptor.java index 6e6b27c1c..c53578bfc 100644 --- a/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/servlet/HttpSessionAdaptor.java +++ b/bundles/org.eclipse.equinox.http.servlet/src/org/eclipse/equinox/http/servlet/internal/servlet/HttpSessionAdaptor.java @@ -274,6 +274,38 @@ public class HttpSessionAdaptor implements HttpSession, Serializable { controller.removeActiveSession(session); } + public void invokeSessionListeners (List<Class<? extends EventListener>> classes, EventListener listener) { + if (classes == null) { + return; + } + + for (Class<? extends EventListener> clazz : classes) { + if (clazz.equals(HttpSessionListener.class)){ + HttpSessionEvent sessionEvent = new HttpSessionEvent(this); + HttpSessionListener httpSessionListener = (HttpSessionListener) listener; + httpSessionListener.sessionDestroyed(sessionEvent); + } + + if (clazz.equals(HttpSessionBindingListener.class) || clazz.equals(HttpSessionAttributeListener.class)) { + Enumeration<String> attributeNames = getAttributeNames(); + while (attributeNames.hasMoreElements()) { + String attributeName = attributeNames.nextElement(); + HttpSessionBindingEvent sessionBindingEvent = new HttpSessionBindingEvent(this, attributeName); + + if (clazz.equals(HttpSessionBindingListener.class)) { + HttpSessionBindingListener httpSessionBindingListener = (HttpSessionBindingListener) listener; + httpSessionBindingListener.valueUnbound(sessionBindingEvent); + } + + if (clazz.equals(HttpSessionAttributeListener.class)) { + HttpSessionAttributeListener httpSessionAttributeListener = (HttpSessionAttributeListener) listener; + httpSessionAttributeListener.attributeRemoved(sessionBindingEvent); + } + } + } + } + } + /**@deprecated*/ public void putValue(String arg0, Object arg1) { setAttribute(arg0, arg1); |