Skip to content

Conversation

@goto-bus-stop
Copy link
Member

After this, Redis will only be used for:

  • disconnected-ness state
  • online users
  • guest count. this should perhaps just use a state variable!
  • pub/sub. this should perhaps just use an event emitter!

The latter two were done that way to theoretically support a multi-node setup, but that was never really supported by other parts of the code, even less so now that sqlite is the main database.

@coveralls
Copy link

coveralls commented Dec 14, 2025

Coverage Status

coverage: 85.951% (-0.2%) from 86.187%
when pulling 26ebf5c on sqlite-sessions
into 96ae461 on default.

@goto-bus-stop goto-bus-stop enabled auto-merge (squash) December 14, 2025 10:50
@goto-bus-stop goto-bus-stop merged commit 1143cae into default Dec 14, 2025
7 checks passed
@goto-bus-stop goto-bus-stop deleted the sqlite-sessions branch December 14, 2025 10:52
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.

3 participants