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

Handle connection closure #19

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

OscarFKE
Copy link
Contributor

Handle the connection to the IRC server closing by cleanly closing DB connection and exiting the process. Changed scheduled event deactivation logic as events were not being reactivated on rejoining a room. Added kick handler to deactivate scheduled events for the channel mockturtle was kicked from.

@pushcx
Copy link
Member

pushcx commented Sep 22, 2024

@OscarFKE and I talked through this on IRC, we're going to try a simpler approach in #20 and hang onto this in case anything requires the manual tidying.

@OscarFKE OscarFKE force-pushed the issue-17-recover-from-disconnects branch from 4dea098 to e89f40e Compare September 23, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants