Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 356726

Summary:	Instead of the sessionDestroyed called sessionCreated after invalidate session
Product:	[RT] Jetty	Reporter:	cerh <cerh>
Component:	server	Assignee:	Jan Bartel <janb>
Status:	RESOLVED FIXED	QA Contact:
Severity:	blocker
Priority:	P3	CC:	gregw, janb, jetty-inbox, linuxhippy, ph
Version:	7.5.0
Target Milestone:	7.5.x
Hardware:	PC
OS:	Linux
Whiteboard:

Description cerh

2011-09-05 08:14:07 EDT

Build Identifier: 7.5.0.v20110901

public abstract class AbstractSessionManager extends AbstractLifeCycle implements SessionManager
{
....
public void removeSession(AbstractSession session, boolean invalidate)
    {
        ...
            if (invalidate && _sessionListeners!=null)
            {
                HttpSessionEvent event=new HttpSessionEvent(session);
                for (HttpSessionListener listener : _sessionListeners)
                    listener.sessionCreated(event); <--!!!!!!!!!!
            }
        }
    }
...
}

Reproducible: Always

Comment 1 cerh

2011-09-05 09:36:28 EDT

And in 8.0.X version

Comment 2 Philip Helger

2011-09-05 11:26:48 EDT

Importance must be "blocker"!!!!

Comment 3 Jan Bartel

2011-09-05 19:16:26 EDT

Fixed for 7.5.1, including updated junit test to check listeners called correctly.

Comment 4 Greg Wilkins

2011-09-05 20:18:17 EDT

argh!!! better get a unit test for this also!

Comment 5 Linuxhippy

2011-09-07 16:29:20 EDT

and I just had a patch prepared for this :/