FAForever Forums
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Groups
    • Login
    The current pre-release of the client ("pioneer" in the version) is only compatible to itself. So you can only play with other testers. Please be aware!

    Ejected: Invalid Player Data

    Scheduled Pinned Locked Moved FAF support (client and account issues)
    11 Posts 5 Posters 713 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • B Offline
      BJ
      last edited by

      Hey

      After coming back from easter break, whenever I try to join a custom game the game launches as normal but as soon as the lobby loads I'm ejected with the message "Invalid Player Data"
      I tried clearing the cache, disabling all game modifications etc. but to no avail.
      I have still been able to join matchmaking and was able to play a game there with no problems.
      If I create a lobby myself it doesn't occurr.

      Something changed over Easter I had never seen this problem before that.

      1 Reply Last reply Reply Quote 0
      • Ctrl-KC Offline
        Ctrl-K
        last edited by Ctrl-K

        I doubt you will ever join then since this is a protection from malicious players trying to break lobbies. Your nickname is having not allowed amount of characters in it, thats why πŸ˜›

        β€œBe a yardstick of quality. Some people aren’t used to an environment where excellence is expected.”
        β€” Steve Jobs.
        My UI Mods
        Support me

        1 Reply Last reply Reply Quote 0
        • B Offline
          BJ
          last edited by

          Why is that all of a sudden an issue ?
          I have been using this nickname on FAF since I joined which is almost 8 years without issue.
          Why does this not occur on matchmaking then ?

          1 Reply Last reply Reply Quote 0
          • B Offline
            BJ
            last edited by

            Found the offending commit to fa
            https://github.com/FAForever/fa/commit/5ccc5547ca10f44037fc6d7c73a4f73a9ea327cb
            @Jip is responsible, where do these guards come from ? What are they guarding against ?

            1 Reply Last reply Reply Quote 0
            • JipJ Offline
              Jip
              last edited by Jip

              These guards exists to (somewhat) sanitize the values we receive in the lobby. As I mentioned on Github, we can* always extend this in the future. Not sure if that was supposed to be taken literally, haha.

              I'd personally recommend to just change your name. But for the sake of purpose, @Sheikah what are the minimum and maximum length of a username? If I'm not mistaken it can be 3 to 32 characters.

              A work of art is never finished, merely abandoned

              1 Reply Last reply Reply Quote 0
              • S Offline
                Sheikah
                last edited by

                Yes those are the character limits

                B 1 Reply Last reply Reply Quote 1
                • B Offline
                  BJ @Sheikah
                  last edited by

                  @jip said in Ejected: Invalid Player Data:

                  These guards exists to (somewhat) sanitize the values we receive in the lobby. As I mentioned on Github, we can* always extend this in the future. Not sure if that was supposed to be taken literally, haha.

                  I'd personally recommend to just change your name. But for the sake of purpose, @Sheikah what are the minimum and maximum length of a username? If I'm not mistaken it can be 3 to 32 characters.

                  Sanitize for what ?
                  The discussion you linked on github talks about to long strings being an attack vector that crashes lobbies but why are we then also introducing a lower bound?

                  Like I said I have a history with my username, on FAF I've been using it for almost 8 years now, without issue I might add, and elsewhere for a decade or more.
                  This doesn't seem to caue any problems with ladder games, nor me making my own lobbies.
                  The lower bound of 3 seems arbitrary, fix no problems, and introduced to no actual benefit.

                  @sheikah said in Ejected: Invalid Player Data:

                  Yes those are the character limits

                  My question is why have these limits been set?
                  More specifically why the lower limit?
                  What issues arrise from my username ?

                  1 Reply Last reply Reply Quote 0
                  • Brutus5000B Offline
                    Brutus5000 FAF Server Admin
                    last edited by Brutus5000

                    The idea of the minimum username length is that users should have at least a meaningful syllable.
                    This requirement was introduced 7 years ago for the registration, so affecting new users. However, 100k+ accounts already existed back then.

                    We have 162 semi-active accounts with less than 3 characters, so this game change should be rolled back @Jip

                    He said, "I've been to the year 3000
                    Not much has changed, but they live underwater
                    And your great-great-great-granddaughter
                    Is playin' FAF, playin' FAF"

                    1 Reply Last reply Reply Quote 0
                    • JipJ Offline
                      Jip
                      last edited by

                      I'll include it with the hotfix then πŸ‘

                      A work of art is never finished, merely abandoned

                      1 Reply Last reply Reply Quote 1
                      • B Offline
                        BJ
                        last edited by

                        @Jip @Sheikah
                        I would like to apologize if my previous response was to confrontational I was pretty irrate and being unable to reach for FAF to blow of some steam didn't help. :Þ
                        I'm glad to hear this will be rolled back and look forward to returning tothe game once the hotfix drops.

                        1 Reply Last reply Reply Quote 0
                        • JipJ Offline
                          Jip
                          last edited by

                          This should now be fixed πŸ‘

                          A work of art is never finished, merely abandoned

                          1 Reply Last reply Reply Quote 0
                          • First post
                            Last post