GitHub PushEvent at 2014-03-12 14:14:28
Repository: aerogear-simplepush-server
Actor: danbev
Payload: {"shas":[["6b9bf8cfe8b35ae54fc24f7ff6ee144787fa405a","ae63d1d4d7994483a3e520cafec7a829fe25c0a1@gmail.com","Fixing SockJS session handling for xhr/jsonp polling.\n\nThe is a re-opening of AGSMPLPUSH-42 which intended to fix this issue\nbut the fix was just plain wrong.\n\nThe original issue was that the polling transports did to reply with any\ndata when messages had been sent to them. The fix then was to add an\nexplicit empty reply to trigger SockJS to repoll and it was then able\nto retrieve data.\nBut the polling transports are \"long polling\" in that they will remain\nuntil either data is made available by the server side or the session\ntimes out.\n\nThis commit fixes the session handling by introducing a field in the\nSockJSSession class which is the ChannelHandlerContext used for an\nopen connection. This 'openContext' is then available to be used\nby SessionState implementations, for example the PollingSessionState.\nPollingSessionState sometimes needs to use the ChannelHandlerContext\nused to connect which is needed to send the SockJS OpenFrame, but\nsuccessive calls needs to use the ChannelHandlerContext for the\nopen session.\n\nAdditional changes:\n- SockJS test server can now be started with mvn exec:java\n- sockjs-client.html contains examples of specifying transports types\n\n[https://issues.jboss.org/browse/AGSMPLPUSH-42]","Daniel Bevenius",false],["47e4aff56c81017b265ab30d03e32f077b35c376","755f736914cdfef95a26c5f750b637288560116b@apache.org","Merge branch 'SockJS'","Matthias Wessendorf",true]],"size":2,"ref":"refs/heads/master","head":"47e4aff56c81017b265ab30d03e32f077b35c376"}