TMM: Add match confirmation (ready check) for matchmaker games

[Askaholic]

For proposed protocol specification see: FAForever/server#607 and FAForever/server#608. The latter PR contains a feature complete implementation that is ready to be tested with a supporting client.

A feature which has already been requested a few times for 1v1 and will be especially relevant for bigger matchmaker games is having a "ready" check before the game starts. The server would send a match_info message which would trigger a popup in the client that looks something like:

Whenever someone clicks the ready button, a new match_info message will be sent so that the client can update the UI. Once everyone accepts the match, the launch sequence will happen as normal. If the timeout is reached before everyone accepts, then the match cancellation sequence will happen and the party will be entered back into the queue.

Wanna have the bug fixed quickly?
Visit Issue hunt...

[RoxyBananaSmoothie]

Hi there, appreciate you guys probably already have a long list of things to fix. I just want to say that this "ready to play" function is absolutely essential for the 2 v 2 ladder. We can't really expect people to stay and wait in the queue for hours without leaving the computer. My friend was in a situation that he was in a queue for a few hours, and the game launched while he was away from his desk. His team mate somehow didn't realise he was afk for 4..5 mins into the game (I know it's strange that he didn't notice within the first minute). Now he blocked my friend and threaten to report him...

[bukajsytlos]

If you leave your PC, you should leave queue too.

…

On Mon, 2 Aug 2021, 15:23 RoxyBananaSmoothie, ***@***.***> wrote: Hi there, appreciate you guys probably already have a long list of things to fix. I just want to say that this "ready to play" function is absolutely essential for the 2 v 2 ladder. We can't really expect people to stay and wait in the queue for hours without leaving the computer. My friend was in a situation that he was in a queue for a few hours, and the game launched while he was away from his desk. His team mate somehow didn't realise he was afk for 4..5 mins into the game (I know it's strange that he didn't notice within the first minute). Now he blocked my friend and threaten to report him... — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1783 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACBYJJNGBHOEK2PRKCOLHK3T22L5DANCNFSM4OCFKOAA> .

[RoxyBananaSmoothie]

Technically, yes. In reality, you can easily be on your PC and still miss the launch if you have been in a queue for hours and forgotten about it. That's easily done in my opinion.

[Katharsas]

Current status:
Implemented in server PR FAForever/server#608 as described in FAForever/server#607 .
With that server PR, the server will only start a game with game_launch when all players have sent match_ready, and the client cannot send match_ready without the PR because that would close the connection. Therefore we need a two stage implementation on the server:

• Server dummy implementation: Server accepts match_ready by client but ignores it and still starts game as usual (see Add dummy implementation for client command match_ready (#607) server#820).
• Client dummy implementation: Auto-send match_ready on match_found to not confuse users, make sure client is ok with receiving match_info
• New server version with server dummy PR is released and users have been forced to upgrade to client version with client dummy
• Server dummy implementation is replaced with PR.
• Full client UI and reacting to match_info.

[Sheikah45]

There is a 4th step as the client implementation in 2 shouldn't have any UI and should just auto send ready because that would just be confusing to the end user.

So the 4th step is implement the UI and real responses from the client.