{"action":"created","issue":{"url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/55","labels_url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/55/labels{/name}","comments_url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/55/comments","events_url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/55/events","html_url":"https://github.com/pac4j/vertx-pac4j/issues/55","id":119964196,"number":55,"title":"User in vertx-web.session is null","user":{"login":"HendrikLanghammer","id":15124162,"avatar_url":"https://avatars.githubusercontent.com/u/15124162?v=3","gravatar_id":"","url":"https://api.github.com/users/HendrikLanghammer","html_url":"https://github.com/HendrikLanghammer","followers_url":"https://api.github.com/users/HendrikLanghammer/followers","following_url":"https://api.github.com/users/HendrikLanghammer/following{/other_user}","gists_url":"https://api.github.com/users/HendrikLanghammer/gists{/gist_id}","starred_url":"https://api.github.com/users/HendrikLanghammer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HendrikLanghammer/subscriptions","organizations_url":"https://api.github.com/users/HendrikLanghammer/orgs","repos_url":"https://api.github.com/users/HendrikLanghammer/repos","events_url":"https://api.github.com/users/HendrikLanghammer/events{/privacy}","received_events_url":"https://api.github.com/users/HendrikLanghammer/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-12-02T15:19:10Z","updated_at":"2016-01-05T09:54:12Z","closed_at":"2015-12-23T12:34:09Z","body":"I always null for the webUser in a SocksJSHandler BridgeEvent: \r\n\r\nSockJSHandler sockJSHandler = SockJSHandler.create(vertx);\r\nsockJSHandler.bridge(options, be -> {\r\n  if (be.socket().webUser() == null) {\r\n    // no active session, no game\r\n    be.complete(false);\r\n...\r\n\r\nI think this is because the UserHolder for the vertx session does not get filled.\r\n\r\nI worked around this issue by using be.socket().webSession().data().get(Pac4jConstants.USER_PROFILE) to get the Pac4j profile instead of using be.socket().webUser()\r\n\r\nUnfortunately this issue breaks the vertx-web documentation when using Pac4jAuthProvider."},"comment":{"url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/comments/168954192","html_url":"https://github.com/pac4j/vertx-pac4j/issues/55#issuecomment-168954192","issue_url":"https://api.github.com/repos/pac4j/vertx-pac4j/issues/55","id":168954192,"user":{"login":"millross","id":1318862,"avatar_url":"https://avatars.githubusercontent.com/u/1318862?v=3","gravatar_id":"","url":"https://api.github.com/users/millross","html_url":"https://github.com/millross","followers_url":"https://api.github.com/users/millross/followers","following_url":"https://api.github.com/users/millross/following{/other_user}","gists_url":"https://api.github.com/users/millross/gists{/gist_id}","starred_url":"https://api.github.com/users/millross/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/millross/subscriptions","organizations_url":"https://api.github.com/users/millross/orgs","repos_url":"https://api.github.com/users/millross/repos","events_url":"https://api.github.com/users/millross/events{/privacy}","received_events_url":"https://api.github.com/users/millross/received_events","type":"User","site_admin":false},"created_at":"2016-01-05T09:54:12Z","updated_at":"2016-01-05T09:54:12Z","body":"I still need to update the vertx-pac4j README, I will make a note to reference use of the VertxProfileManager rather than just the default ProfileManager implementation if using a profile manager in client code.\n\n\n> On 5 Jan 2016, at 07:13, HendrikLanghammer <notifications@github.com <mailto:notifications@github.com>> wrote:\n> \n> Of course it's working perfectly using the VertxProfileManager - thank you!\n> \n> —\n> Reply to this email directly or view it on GitHub <https://github.com/pac4j/vertx-pac4j/issues/55#issuecomment-168921021>.\n> \n\n"}}