Minor polish and FIXMEs
This commit is contained in:
parent
ff2e9aa5bc
commit
db4de526d2
|
@ -104,7 +104,7 @@ public class SockJsWebSocketHandler implements TextMessageHandler {
|
|||
Assert.notNull(wsSession, "wsSession is required");
|
||||
String path = wsSession.getURI().getPath();
|
||||
String[] segments = StringUtils.tokenizeToStringArray(path, "/");
|
||||
Assert.isTrue(segments.length > 3, "SockJS request should have at least 3 patgh segments: " + path);
|
||||
Assert.isTrue(segments.length > 3, "SockJS request should have at least 3 path segments: " + path);
|
||||
return segments[segments.length-2];
|
||||
}
|
||||
|
||||
|
|
|
@ -43,6 +43,7 @@ public interface RequestUpgradeStrategy {
|
|||
* @param handler the handler for WebSocket messages
|
||||
*/
|
||||
void upgrade(ServerHttpRequest request, ServerHttpResponse response, String selectedProtocol,
|
||||
HandlerProvider<WebSocketHandler> handler) throws Exception;
|
||||
HandlerProvider<WebSocketHandler> handlerProvider) throws Exception;
|
||||
// FIXME how to indicate failure to upgrade?
|
||||
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue