cloneRequest puts copied session in wrong object on Tomcat 7

[fabiangebert]

AtmosphereRequest#cloneRequest puts the fake session in the session builder:

.session(copySession ? new FakeHttpSession(request.getSession(true)) : null);

In the AtmosphereRequest$Builder this is executed:

if (NoOpsRequest.class.isAssignableFrom(request.getClass())) {
    NoOpsRequest.class.cast(request).fake = session;
} else {
   hackedJettySession = session;
}

And since at the time of call, the request is no NoOpsRequest, the session will be put in hackedJettySession.

The only way I can get the session now is the following (Groovy):
value = cometResource.request.b.hackedJettySession.getAttribute(key)

This works. However, on Jetty, this works, too:
value = cometResource.session.getAttribute(key)

It would be nice to have it this simple on Tomcat, too.

[jfarcand]

Agree. Looking.