osu-server-spectator
osu-server-spectator copied to clipboard
ppy
osu! spectator server
Found while testing https://github.com/ppy/osu/pull/28119. Please read diatribe attached to test case for explanation. "Fortunately" this does not affect replays being recorded right now because by sheer luck the only thing...
> [!WARNING] > This - or an alternative PR fixing this same issue - **must** be merged before the next bump of game packages in spectator server. Without fixing this...
``` $countryAcronym = $_SERVER["HTTP_CF_IPCOUNTRY"]; $conn->exec("INSERT INTO user_country_history (user_id, `year_month`, country_acronym) SELECT $userId, date_format(CURRENT_TIMESTAMP, '%y%m'), '{$countryAcronym}' ON DUPLICATE KEY UPDATE count = count + 1"); ```
- [x] Depends on https://github.com/ppy/osu-server-spectator/pull/214 for mergeability - [x] Ask @ThePooN if he's okay with deploying this This has been bothering me for a little while. Spectator server logging is...
We need a way to stop older clients from connecting. This is already possible on score token retrieval (via checks `osu-web` side) but we don't have a flow in place...
We should likely be tracking which instance is responsible for rooms at a database level, and perform cleanup on non-running instances (when a new instance is spawned). This will cover...
Disregarding all discussion above, a first point to address is the ability for a user to reconnect to `osu-server-spectator`. Regardless of whether we move responsibilities around, we need to ensure...
In theory, we should be able to do something like this: ```diff diff --git a/osu.Game/Screens/Play/SubmittingPlayer.cs b/osu.Game/Screens/Play/SubmittingPlayer.cs index 5fa6508a31..663ea9ab81 100644 --- a/osu.Game/Screens/Play/SubmittingPlayer.cs +++ b/osu.Game/Screens/Play/SubmittingPlayer.cs @@ -131,7 +131,8 @@ protected override async...
@ThePooN discovered during some infra work that a Redis connection failure will bring down spectator flows completely with  As is, redis definitely wasn't supposed to be *this* service-critical as...
This feature spans across osu, osu-server-spectator and osu-web projects. This should be merged only when all PRs are approved. - [ ] Depends on (ppy/osu/pull/23510) # Changes This PR just...
