Test: Re-subscribe on re-connect #41
Labels
No labels
bug
confirmed
critical
discussion
documentation
Doing
enhancement
suggestion
support
To Do
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: hazaar/hazaar-warlock#41
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Something I need to test and fix if it's actually an issue. A client will connect to the server, then they may be disconnected for some reason at which point it is possible that any subscriptions are deleted by the server if it too registers the disconnect. Or even, if the server restarts and the client automatically reconnects.
The issue here is, do we automatically re-subscribe to any subscriptions the client had previously requested? if not, this is something we will need to fix up.
This was actually almost already implemented. The reason it wasn't working was because the re-subscribe only happened when loading the initial packet, which didn't happen on a reconnect because we already knew the packet types. So now we do a different test for the initial packet which allows us to re-subscribe. We can also reload the packet types in case they change as well.
mentioned in commit
35b1077af6