| Summary: | No CloseMessage because of WebSocket connection has been closed | ||
|---|---|---|---|
| Product: | Tomcat 8 | Reporter: | Artem Bilan <abilan> |
| Component: | WebSocket | Assignee: | Tomcat Developers Mailing List <dev> |
| Status: | RESOLVED FIXED | ||
| Severity: | major | CC: | vstorm83 |
| Priority: | P2 | ||
| Version: | 8.0.32 | ||
| Target Milestone: | ---- | ||
| Hardware: | PC | ||
| OS: | All | ||
I've added the referenced test case to 9.0.x and 8.0.x. It passes with 9.0.x but not 8.0.x. I'm looking at a fix now. Fixed in 8.0.x and 7.0.x. The fix will be included in 8.0.33 and 7.0.69 onwards. Thank you, Mark, for the fix! I would like to know if I can test it from my perspective against the 8.0.33 BUILD-SNAPSHOT. From other side let us know when you are going to release those maintenance versions. There are no automatic snapshots although we can trigger one manually if that would be easier than you building from source. 8.0.x releases on a roughly monthly release cycle. I tend to start preparation at the start of the month with the release being generally available some time in the second week. Fair enough, Mark! We definitely can wait for middle of March for the release. That is fully upfront in time with all our Spring release train in the May around Spring Boot 1.4. Meanwhile I really would like to ask you about BUILD-SNAPSHOT. Not sure that check out the full Tomcat source code and build it locally would be easier for me :-). Thank you again! It has been a while since I ran the snapshot publishing script. It is currently running and looks OK so far. Give it about 15 mins and you should have a complete 8.0.x snapshot of the current trunk under: https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/ Let us know how you get on with it. Well, Mark, looks like we are really good with the fix: :spring-integration-websocket:compileTestJava Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-websocket/8.0-SNAPSHOT/tomcat-embed-websocket-8.0-20160223.155055-9.pom Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-core/8.0-SNAPSHOT/tomcat-embed-core-8.0-20160223.154911-17.pom Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-websocket/8.0-SNAPSHOT/tomcat-embed-websocket-8.0-20160223.155055-9.jar Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-core/8.0-SNAPSHOT/tomcat-embed-core-8.0-20160223.154911-17.jar :spring-integration-websocket:processTestResources :spring-integration-websocket:testClasses :spring-integration-websocket:testAll Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-logging-log4j/8.0-SNAPSHOT/tomcat-embed-logging-log4j-8.0-20160223.155143-17.pom Download https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/embed/tomcat-embed-logging-log4j/8.0-SNAPSHOT/tomcat-embed-logging-log4j-8.0-20160223.155143-17.jar BUILD SUCCESSFUL Total time: 1 mins 11.597 secs Thank you very much! Cheers, Artem |
In our test-cases we would like to handle Endpoint.onClose(Session session, CloseReason closeReason) {} But looks the code isn't reachable. WsSession does: state = State.CLOSED_SENT; ... sendCloseMessage(closeReasonMessage); Where we have the code like in WsRemoteEndpointImplBase.startMessageBlock: FutureToSendHandler f2sh = new FutureToSendHandler(wsSession); startMessage(opCode, payload, last, f2sh); try { if (timeout == -1) { f2sh.get(); } else { f2sh.get(timeout, TimeUnit.MILLISECONDS); } That f2sh.get() does this: wsSession.registerFuture(this); where: if (isOpen()) { futures.put(f2sh, f2sh); } is false because our state is State.CLOSED_SENT not like: state == State.OPEN || state == State.SENDING_CLOSED Therefore we never get result from that Future and our onClose() test fail. You can find more info in the bug57489, especially https://bz.apache.org/bugzilla/show_bug.cgi?id=58624#c13 and its attachment. The related Spring Integration JIRA: https://jira.spring.io/browse/INT-3909. Thank you in advance!