Skip to content

Eagerly connect to latest server instance for best online experience#37506

Open
peppy wants to merge 5 commits intoppy:masterfrom
peppy:better-graceful-shutdown
Open

Eagerly connect to latest server instance for best online experience#37506
peppy wants to merge 5 commits intoppy:masterfrom
peppy:better-graceful-shutdown

Conversation

@peppy
Copy link
Copy Markdown
Member

@peppy peppy commented Apr 24, 2026

Client side requirements for making the client connect as soon as possible, based on how the client is being used. This is especially important with the introduction of ranked play: previously the worst case scenario would be that you couldn't join a multiplayer room (or spectate a user) and this was automatically handled mostly*, but now, if you leave the game open for a while, you can potentially be stuck queueing in ranked play with no users remaining on your server.

Some samples of how this looks follow. Do note that the client is showing "Server is shutting down" errors. This is only going to happen in local debug environments – In production, when you reconnect to the endpoint you will always get a non-shutting-down instance.

Idle scenario:

osu.2026-04-24.at.08.01.38.mp4

Non-idle scenario:

osu.2026-04-24.at.09.02.33.mp4
  • Spectator isn't handled properly, as one example.

@peppy peppy added the area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. label Apr 24, 2026
@peppy peppy force-pushed the better-graceful-shutdown branch from af2b659 to 258e4d0 Compare April 24, 2026 09:07
peppy added 3 commits April 24, 2026 18:08
The `Disconnect` method, while it might look correct on the surface, is
actually a forceful disconnect which logs out of the API too..
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant