{"action":"created","issue":{"url":"https://api.github.com/repos/scalatra/scalatra/issues/429","labels_url":"https://api.github.com/repos/scalatra/scalatra/issues/429/labels{/name}","comments_url":"https://api.github.com/repos/scalatra/scalatra/issues/429/comments","events_url":"https://api.github.com/repos/scalatra/scalatra/issues/429/events","html_url":"https://github.com/scalatra/scalatra/issues/429","id":46692517,"number":429,"title":"AtmosphereHandler.onDisconnect needs some rework","user":{"login":"dozed","id":239058,"avatar_url":"https://avatars.githubusercontent.com/u/239058?v=3","gravatar_id":"","url":"https://api.github.com/users/dozed","html_url":"https://github.com/dozed","followers_url":"https://api.github.com/users/dozed/followers","following_url":"https://api.github.com/users/dozed/following{/other_user}","gists_url":"https://api.github.com/users/dozed/gists{/gist_id}","starred_url":"https://api.github.com/users/dozed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dozed/subscriptions","organizations_url":"https://api.github.com/users/dozed/orgs","repos_url":"https://api.github.com/users/dozed/repos","events_url":"https://api.github.com/users/dozed/events{/privacy}","received_events_url":"https://api.github.com/users/dozed/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/scalatra/scalatra/labels/atmosphere","name":"atmosphere","color":"fad8c7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":15,"created_at":"2014-10-24T00:16:57Z","updated_at":"2015-02-10T13:18:40Z","closed_at":null,"body":"Two problems:\r\n\r\n1. a closing AtmosphereResource can close the HttpSession (invalidate, code already commented out)\r\n2. a closing AtmosphereResource can remove the AtmosphereClient reference from the HttpSession\r\n\r\nThis may require a better handling."},"comment":{"url":"https://api.github.com/repos/scalatra/scalatra/issues/comments/73697297","html_url":"https://github.com/scalatra/scalatra/issues/429#issuecomment-73697297","issue_url":"https://api.github.com/repos/scalatra/scalatra/issues/429","id":73697297,"user":{"login":"wkozaczuk","id":5296673,"avatar_url":"https://avatars.githubusercontent.com/u/5296673?v=3","gravatar_id":"","url":"https://api.github.com/users/wkozaczuk","html_url":"https://github.com/wkozaczuk","followers_url":"https://api.github.com/users/wkozaczuk/followers","following_url":"https://api.github.com/users/wkozaczuk/following{/other_user}","gists_url":"https://api.github.com/users/wkozaczuk/gists{/gist_id}","starred_url":"https://api.github.com/users/wkozaczuk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wkozaczuk/subscriptions","organizations_url":"https://api.github.com/users/wkozaczuk/orgs","repos_url":"https://api.github.com/users/wkozaczuk/repos","events_url":"https://api.github.com/users/wkozaczuk/events{/privacy}","received_events_url":"https://api.github.com/users/wkozaczuk/received_events","type":"User","site_admin":false},"created_at":"2015-02-10T13:18:40Z","updated_at":"2015-02-10T13:18:40Z","body":"Regarding patch - https://github.com/scalatra/scalatra/commit/b13970bf4b6b10866497da79e1f39e23566c20c0 - it would fix one of my problems but I would have to reimplement the ClientFilter. My take on it is that this patch does change the api of the frameworks and breaks compatibility."}}