-
Notifications
You must be signed in to change notification settings - Fork 7
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
Solid WebSockets API: Missing Unsubscribe! #6
Comments
Try reviewing this maybe? |
@MasterJames I am a bit confused by your comment! Do you: |
Maybe a little from column A (not knowing the state of affairs) and a little from column B (if it is not working properly in the latest builds, or yes to add comment/feedback to the ideas [? If what sould be obvious is not] to your liking or before it gets more engrained if not fully happening yet). |
I have looked the spec and the NSS code! The WebSocket API needs a unsubscribe call! |
I think, a naive API would be to return a function of the subscribe call which allows to unsubscribe. However, the removeListener API for Socket.io looks a bit different. |
IMHO you do not need a complicated API here. Whatever the API, the server will ultimately need to see a message on the websocket (since the socket only carries messages), something of the form |
Moving this issue to solid/specification for consideration. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I believe that the current draft defines unidirectional notifications. The resource is specified as topic during subscription and all the further communication is unidirectional. When it comes to unsubscribing, I see why it is needed for subscription types where the publisher delivers a notification to target specified during the subscription step. For subscription type where subscriber connects to notifications source, closing connection seems sufficient. Why do you see the need for specific unsubscribe request? |
@elf-pavlik The only reason I am reluctant to close this issue is that there are items linked here (see links brought in by @csarven here and in turn in those issue), which form a tangled web. I am happy to continue discussion in #145 and keep the discussion in one place, but I propose closing this along with #145, |
How does one unsubscribe to watching a resource without terminating the websocket connection itself?
The text was updated successfully, but these errors were encountered: