Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convergence Server does not complete web socket close handhsake. #218

Closed
mmacfadden opened this issue Jun 10, 2021 · 1 comment
Closed
Assignees
Labels
bug Something isn't working server Issues related to the server
Milestone

Comments

@mmacfadden
Copy link
Contributor

Versions

  • Convergence Version: 1.0.0-rc.8

Describe the Bug
When the javascript client tries to close the connection, the close process does not complete cleanly, and often waits for a timeout. This is because the server is not sending back the web socket close FIN message. It is supposed to work as follows:

Client.close() -> Web Socket Close [code, message] -> Server ->  Server Close Event Fired
                                                       |
Client <- Web Socket Close[same code, same message] <--|
  |
  |--> Client Close Event Fired

Without the server sending the TCP FIN message back to the client, the client will wait until whatever timeout the specific implementation provides. This often results in a protracted delay, as well as an error event being emitted.

Expected Behavior
The server should properly complete the close request.

@mmacfadden mmacfadden added the bug Something isn't working label Jun 10, 2021
@mmacfadden mmacfadden added this to the 1.0.0-rc.9 milestone Jun 10, 2021
@mmacfadden mmacfadden self-assigned this Jun 10, 2021
@mmacfadden
Copy link
Contributor Author

This is related to the following Akka HTTP issue.

akka/akka-http#1358

The key information being in this comment: akka/akka-http#1358 (comment)

We need to update the server to use fromSinkAndSourceCoupled instead of fromSinkAndSource in the WebSocketService.

@mmacfadden mmacfadden added the server Issues related to the server label Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working server Issues related to the server
Projects
None yet
Development

No branches or pull requests

1 participant