HttpSessionListener.sessionCreated() not called

Description

HttpSessionListener.sessionCreated() is not called by Pax Web 4.0.0-SNAPSHOT, which is a regression from 3.0.4.

This breaks org.ops4j.pax.cdi.test.ServletTest.servletInjectionWithSessionScope() on the Pax CDI pax-web-4 branch:

The same test passes on master with Pax Web 3.0.4. Here is a log excerpt from the working version:

The "session created" message is missing with Pax Web 4.0.0-SNAPSHOT.

Environment

None

Activity

Show:
Achim Nierbeck
August 3, 2014, 7:43 PM
Edited

The Listeners are registered by the HttpContext while they are called by the SessionManager which doesn't find a single listener, this seems to be the gap.

Achim Nierbeck
August 13, 2014, 10:42 PM

Branch: refs/heads/master
Home: https://github.com/ops4j/org.ops4j.pax.web
Commit: 4e8c2c5ceda7fdc2b2c1c641977787f0aaeb83aa
https://github.com/ops4j/org.ops4j.pax.web/commit/4e8c2c5ceda7fdc2b2c1c641977787f0aaeb83aa
Date: 2014-08-14 (Thu, 14 Aug 2014)

Achim Nierbeck
August 14, 2014, 6:05 PM

needs to be fixed also for 3.1.x

Achim Nierbeck
August 14, 2014, 7:15 PM

Branch: refs/heads/pax-web-3.1.x
Home: https://github.com/ops4j/org.ops4j.pax.web
Commit: 1811c1b47972f7cdc18ede22a441028b49bd6cbf
https://github.com/ops4j/org.ops4j.pax.web/commit/1811c1b47972f7cdc18ede22a441028b49bd6cbf
Date: 2014-08-14 (Thu, 14 Aug 2014)

Changed paths:
M pax-web-jetty/src/main/java/org/ops4j/pax/web/service/jetty/internal/HttpServiceContext.java

Assignee

Achim Nierbeck

Reporter

Harald Wellmann

Labels

None

Components

Fix versions

Affects versions

Priority

Major
Configure