SmartlyDressedGames / Unturned-3.x-Community

Community portion of the Unturned-3.x repo. If you have access to the source code you can find it here:
https://github.com/SmartlyDressedGames/Unturned-3.x/
84 stars 18 forks source link

askVehicle Spam Crashes The Server #125

Closed N1GHTPATROL closed 6 years ago

N1GHTPATROL commented 6 years ago

Various hacked clients have a crash server feature which still works after this update.

Kr4ken-9 commented 6 years ago

Can confirm, as I said in #107 , any spammed packet will kill the server.

aviadmini commented 6 years ago

Current implementation of max packets it useless

Instead, one should limit RPCs from client by amount of calls defined per RPC. E.g. askVehicles should be called only once so more calls most likely mean that client is "hacked" (note the quotes)

These "hacked" clients are not even hacked. Unfortunately, more and more people get to know the easy way to "hack" the client

SDGNelson commented 6 years ago

The current max packets implementation is more to prevent spamming in general. These RPCs will have to be reviewed on a case by case basis. For the next patch I'll limit askVehicles to once per connection.

aviadmini commented 6 years ago

Limiting total packets it nonsense.

Exactly, case by case basis BUT with a general rule. I'm pretty sure none of the RPCs should be received 50 times per second.

Of course adding a nuclear exposive chewing gum spitter boss is much more important than working on game stability and hack and so-called-hack prevention

SDGNelson commented 6 years ago

It's not total packets, rather max packets per second per player. I think what you're asking for is what was implemented. Is that the case? I may have miscommunicated the feature.

You sound a bit frustrated, but a balance between content, features and stability is important to maintain interest. For example when my focus was entirely on the editor and performance for several months last year player numbers dropped significantly.

aviadmini commented 6 years ago

I guess my "total packets" is ambiguous, so thanks for clarification. I did mean what you said and jumped to wrong conclusion based on the last update behavior (didn't look in the code because it's quite uncomfortable to decompile each version).

Not sure how features/content can maintain interest when stability is messed up. But you probably know better than me ¯\_(ツ)_/¯

SDGNelson commented 6 years ago

Not sure how features/content can maintain interest when stability is messed up. But you probably know better than me ¯(ツ)

Do you have any particular stability suggestions in mind?

aviadmini commented 6 years ago

Alright,

There's more, might check /r/unturned https://vgy.me/3Sojcf.png - although it needs time to browse through

Hugs <3

SDGNelson commented 6 years ago
N1GHTPATROL commented 6 years ago

Hello Nelson,

Regarding the game not loading on multiplayer (Stuck on Queue Pos 1) there are many issues that can be the cause, including map corruption or more commonly an issue with workshop content. It's also weird because commonly people can hear their surroundings but get to gaze at the queue pos 1 screen.

Involving 4.x servers, I would suggest that you invest a good amount of time to allow servers with 100+ people to function with no problem. This in my opinion would give players more confidence, seeing a server with for example 60/100 rather than 4/24 spread among 15 different servers. As a server owner, I dream of a day in which Unturned servers support hundreds of people, and honestly large servers would spark a whole lot more interest in Unturned, considering it is heavily multiplayer. Those are just my thoughts- please feel free to remark <3

aviadmini commented 6 years ago

No, I don't know which packet(-s) exactly causes it. And I'm not willing to figure it out

As for kick packets - you can ignore that, it was sent by a plugin. However would be nice if it worked

Kr4ken-9 commented 6 years ago

Example of packet spamming in use

Edit: Starts at 32:50