[jira] [Commented] (WICKET-6571) BaseWebSocketBehavior should check sessionId cookie name dynamically

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WICKET-6571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16554389#comment-16554389 ] 

ASF subversion and git services commented on WICKET-6571:
---------------------------------------------------------

Commit 9f3bbac34e4ba914473e04a12b77386b6a2a8186 in wicket's branch refs/heads/master from Martin Tzvetanov Grigorov
[ https://git-wip-us.apache.org/repos/asf?p=wicket.git;h=9f3bbac ]

WICKET-6571 BaseWebSocketBehavior should check sessionId cookie name dynamically


> BaseWebSocketBehavior should check sessionId cookie name dynamically
> --------------------------------------------------------------------
>
>                 Key: WICKET-6571
>                 URL: https://issues.apache.org/jira/browse/WICKET-6571
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket-native-websocket
>    Affects Versions: 8.0.0
>            Reporter: Jan Krakora
>            Assignee: Martin Grigorov
>            Priority: Major
>
> When creating a WebSocket connection, Wicket tries to check if session id is tracked by URL or by cookies.
> The code is in the org.apache.wicket.protocol.ws.api.BaseWebSocketBehavior#getSessionId(Component component)
> It uses a hardcoded sessionId cookie name ({color:#333333}JSESSIONID{color}). So when an application uses a different sessionId cookie name, this code wrongly assumes there is no cookie with session id and appends session Id in the websocket connection URL.
> I think it should use to get session id cookie name
> {code:java}
> servletContext.getSessionCookieConfig().getName(){code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)