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

Server should automatically unadvertise service when advertising client disconnects #684

Closed
jtbandes opened this issue Nov 29, 2021 · 2 comments

Comments

@jtbandes
Copy link
Member

If a client advertises a service, then disconnects, the server still attempts to forward incoming service requests to that client, and then when the service is called it dumps a WebSocketClosedError to the console.

Expected behavior is: when the client disconnects, the server should automatically unadvertise the service and never make future attempts to send messages to the client.

jtbandes added a commit that referenced this issue Nov 29, 2021
**Public API Changes**
None

**Description**
Fixes an issue reported by @Stephenjerry in #650 (comment). When a duplicate service is advertised by the client, the code now correctly shuts down the original service.

There is still a `WebSocketClosedError` received with only a single client, when it advertises a service and then disconnects. Filed #684 to track this.
@github-actions
Copy link

This issue has been marked as stale because there has been no activity in the past 6 months. Please add a comment to keep it open.

@github-actions github-actions bot added the stale label May 29, 2022
@jtbandes jtbandes removed the stale label May 31, 2022
@github-actions
Copy link

github-actions bot commented Jun 1, 2023

This issue has been marked as stale because there has been no activity in the past 12 months. Please add a comment to keep it open.

@github-actions github-actions bot added the stale label Jun 1, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant