Search Unity

  1. Welcome to the Unity Forums! Please take the time to read our Code of Conduct to familiarize yourself with the forum rules and how to post constructively.
  2. We have updated the language to the Editor Terms based on feedback from our employees and community. Learn more.
    Dismiss Notice

Third Party Making Player Movement Secure with Mirror

Discussion in 'Multiplayer' started by Steff00212, Feb 17, 2020.

  1. Steff00212

    Steff00212

    Joined:
    May 17, 2013
    Posts:
    14
    Hey guys,

    I am currently working on a small multiplayer FPS Game using Mirror.
    Since I want it to be secure and as "Cheater-free" as possible, I've searched for a lot of discussions and sources dealing with security in Multiplayer Games using Mirror. Well, turns out there's not very much out there. So I tried dealing with it myself. Let's say it did not work as expected :D

    What I am trying to do, is moving the LocalPlayer, who does not have LocalPlayerAuthority (in order to keep it safe, again), server-side. So I need to process the Input from each Player on the Server and then Update the Position of the Player while sending it to all other Players and the Player who requested the Movement itself. All Players have a NetworkTransform, so actually only updating the Position once should work too. I played around with [Command], [ClientRpc], [Server] and so on. Don't get it working. It does work totally fine if I move the Player locally. But this also gives the Player the possibility to cheat, since you only need to Update your Position locally and the Server then updates it to all other Players. You can find the code I use for Movement below.

    Any tips/tricks are very appreciated, and I hope this thread can help others too!

    Movement Code: https://pastebin.com/40dw5s0k
     
    tigerleapgorge likes this.
  2. mischa2k

    mischa2k

    Joined:
    Sep 4, 2015
    Posts:
    4,333
    NetworkTransform has two modes, either server authoritative (by default), or client authoritative (where players can cheat).

    For server authoritative it's meant to be used like this:
    Code (CSharp):
    1. player sends CmdPleaseMoveMeTo(xyz);
    2. server checks if this is a valid move.
    3. server moves it there
    4. NetworkTransform automatically syncs it to clients
    In your example, you are trying to use server auth movement but you also set the local client's transform.position (instead of letting NetworkTransform set it).

    If you want to set transform.position on the local player, then you need to use NetworkTransform's clientAuthority option - which allows cheating as you know.

    Since you don't want cheating, you are right in that you need to use server authority. In a perfect world, the above code would be just fine. In our world, you have latency. So when you press the 'W' key and ask the server to please move you forward by 1 meter, then it takes way too long for the server to reply. It works, but it feels terrible.

    What you want is either client side prediction, or rubberband movement.

    Client side prediction
    means: ask the server to please move you forward by 1 meter. Then instead of waiting for the reply, take a guess on what the answer will be (probably yes unless there is a wall in front of you), then already move forward hoping that the server ends up with the same end position. This can work. It's not easy to get right for the local player though. There are still latencies and you are still trying to make imperfect predictions which will occasionally be wrong, causing position resets. Note that client side prediction is cheat safe, because the server still doesn't trust the client. The client just makes some assumptions but the server has the last word on it.

    Another option is rubberbanding. This is usually used in MMOs, and I had good success with it in my uMMORPG and uSurvival assets. Unlike client side prediction, you don't ask the server to please move you forward 1 meter and then wait for the result. Instead, you move forward 1 meter when ever you want, then you tell the server 'I moved forward 1 meter', then the server checks if this was a valid move. If it wasn't (if you cheated through a wall, or moved too fast), then the server will reject the move, keep the previously valid position and notify the player that it was rejected. This is still cheat safe. Imho this is the 'easiest' way to do it.

    Neither option is easy. In fact, it's very difficult. As far as I know, the character controller in my assets is the only one for Mirror that does it. And even that one isn't completely finished yet.

    We don't have a client side prediction or a rubberbanding NetworkTransform in Mirror by default, because those depend a lot on your game. For example, you might allow different movement speeds based on different states. Some games have crouching, some have running, some have jumping, some allow movement while jumping where others don't. Some allow vertical climbing. So what move is valid really depends on your game.

    Note that there is a third option, which is what a lot of AAA games / MMOs do. You use the regular NetworkTransform with clientAuthority (where players could cheat), and then you add anti cheat protections to the client. Depending on how successful your game will be, you will need really good anti cheat protection to get away with this. Simple isDebuggerPresent checks won't do, most hackers can bypass those in 5 minutes. Imho you would need at least virtualization with tools like Themida.
    A lot of big games use this method because both client side prediction and rubberbanding are extremely difficult to get right. Whenever you hear about players speedhacking in popular games like GTA, it's because the developers chose this way (the lazy way). Personally I don't like this method. It's understandable why people try to get away with it considering deadlines and budgets, but in the end people will cheat and then you are in trouble.
     
  3. tigerleapgorge

    tigerleapgorge

    Joined:
    Sep 17, 2019
    Posts:
    4
    Very interesting, thank you.
     
  4. ep1s0de

    ep1s0de

    Joined:
    Dec 24, 2015
    Posts:
    164
    You need to send not the player’s coordinates from the client but the key states (see UserCommand - Source Engine)

    And you also need to implement client-side prediction so that you can roll back a local player on the client to a certain state



    You can look at my video...

    Netcode based on RakNet