{"action":"created","issue":{"url":"https://api.github.com/repos/vaadin/cdi/issues/176","repository_url":"https://api.github.com/repos/vaadin/cdi","labels_url":"https://api.github.com/repos/vaadin/cdi/issues/176/labels{/name}","comments_url":"https://api.github.com/repos/vaadin/cdi/issues/176/comments","events_url":"https://api.github.com/repos/vaadin/cdi/issues/176/events","html_url":"https://github.com/vaadin/cdi/issues/176","id":180858117,"number":176,"title":"UIScopedContext leaks closed UIs","user":{"login":"kumm","id":254770,"avatar_url":"https://avatars.githubusercontent.com/u/254770?v=3","gravatar_id":"","url":"https://api.github.com/users/kumm","html_url":"https://github.com/kumm","followers_url":"https://api.github.com/users/kumm/followers","following_url":"https://api.github.com/users/kumm/following{/other_user}","gists_url":"https://api.github.com/users/kumm/gists{/gist_id}","starred_url":"https://api.github.com/users/kumm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kumm/subscriptions","organizations_url":"https://api.github.com/users/kumm/orgs","repos_url":"https://api.github.com/users/kumm/repos","events_url":"https://api.github.com/users/kumm/events{/privacy}","received_events_url":"https://api.github.com/users/kumm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-10-04T10:33:56Z","updated_at":"2016-11-20T20:06:03Z","closed_at":null,"body":"All closed UI is referenced from UIScopedContext storageMap until http session invalidation.\n\nSimple to reproduce, just build a helllo-world vaadin app with one CDIUI.\nRun it, and refresh the page in the browser. Every time you refresh, an UI with closing=true leaked from UIScopedContext. - I know i can use PreserveOnRefresh.\nBut same result when opening the UI in a new tab, closing it, and waiting 3 heartbeatInterval.\n\nI think UIScopedContext should free the UI after it's closed.\n\nHere is a screenshot from jvisualvm. [![vaadincdijvm1.png](https://s11.postimg.org/luw1u4yfn/vaadincdijvm1.png)](https://postimg.org/image/gw8jflumn/)\n\nTested with CDi addon 1.0.3, and vaadin 7.7.2.\n"},"comment":{"url":"https://api.github.com/repos/vaadin/cdi/issues/comments/261801732","html_url":"https://github.com/vaadin/cdi/issues/176#issuecomment-261801732","issue_url":"https://api.github.com/repos/vaadin/cdi/issues/176","id":261801732,"user":{"login":"kumm","id":254770,"avatar_url":"https://avatars.githubusercontent.com/u/254770?v=3","gravatar_id":"","url":"https://api.github.com/users/kumm","html_url":"https://github.com/kumm","followers_url":"https://api.github.com/users/kumm/followers","following_url":"https://api.github.com/users/kumm/following{/other_user}","gists_url":"https://api.github.com/users/kumm/gists{/gist_id}","starred_url":"https://api.github.com/users/kumm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kumm/subscriptions","organizations_url":"https://api.github.com/users/kumm/orgs","repos_url":"https://api.github.com/users/kumm/repos","events_url":"https://api.github.com/users/kumm/events{/privacy}","received_events_url":"https://api.github.com/users/kumm/received_events","type":"User","site_admin":false},"created_at":"2016-11-20T20:06:03Z","updated_at":"2016-11-20T20:06:03Z","body":"@doggy-dev : There are multiple problems in this add-on with destroying contextual instances. #169 is about viewcontext, so not related. This issue is about leaking UIs. \r\nThe pull request #178 for this issue fixes destroying of UIs, not Views ( not even UIScoped beans ).\r\nBut we can say issue #155 is closely related, and fixed by the pull request.\r\n"}}
{"actor":{"display_login":"kumm"}}