RepoMirrors/osu
1
0
Fork 0
mirror of https://github.com/ppy/osu synced 2025-03-23 03:16:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1422 from peppy/fix-api-init-requests

Browse Source 
Fix multiple requests potentially being triggered during connect sequence
This commit is contained in:
Dan Balasescu 2017-10-24 19:52:45 +09:00 committed by GitHub
commit 93f691a487
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
osu.Game/Online/API/APIAccess.cs
View File

@ -121,18 +121,28 @@ namespace osu.Game.Online.API
continue; continue;
} }
var userReq = new GetUserRequest(); var userReq = new GetUserRequest();
userReq.Success += u => userReq.Success += u =>
{ {
LocalUser.Value = u; LocalUser.Value = u;
failureCount = 0;
//we're connected! //we're connected!
State = APIState.Online; State = APIState.Online;
failureCount = 0;
}; };
if (!handleRequest(userReq)) if (!handleRequest(userReq))
{
Thread.Sleep(500);
continue; continue;
}
// The Success callback event is fired on the main thread, so we should wait for that to run before proceeding.
// Without this, we will end up circulating this Connecting loop multiple times and queueing up many web requests
// before actually going online.
while (State != APIState.Online)
Thread.Sleep(500);
break; break;
} }