uklance / tapestry-cometd

A push library for Tapestry5 based on CometD
17 stars 3 forks source link

Investigate atmosphere cometd #28

Closed uklance closed 12 years ago

uklance commented 12 years ago

Atmosphere claims to support all servlet containers and has a CometD servlet using the atmosphere framework. https://github.com/Atmosphere/atmosphere/wiki/Getting-started-with-Cometd http://atmosphere.github.com/atmosphere/apidocs/org/atmosphere/cometd/CometdServlet.html http://atmosphere-framework.2306103.n4.nabble.com/Using-CometD-with-Atmosphere-td4650724.html

jfarcand commented 11 years ago

Salut, nay reason to have stopped using Atmosphere? Please let me know if you face any issue, I will be happy to fix them and help with the integration. Thanks

uklance commented 11 years ago

Hi jfarcand, I was actually trying to find a non-atmosphere solution to get a handle on the current HttpServletRequest. Turns out that this is not available when using a WebSocket. Since I only actually needed the HttpSession (and not the request) I found a solution here.

Looks like I forgot to switch my pom.xml back to use atmosphete-cometd after solving my problem. I'll make sure I switch back before releasing the next version.

How very observant of you... github is the new big-brother ;)