Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Max players are ignored if many players are connecting at the same time #10668

Open
paaulhier opened this issue May 5, 2024 · 1 comment
Open
Labels
status: needs triage type: bug Something doesn't work as it was intended to. version: 1.20.4 Game version 1.20.4

Comments

@paaulhier
Copy link

Expected behavior

If the server reaches the maximum of players, all other players should be disconnected.

Observed/Actual behavior

e.g. i send 4 players on a velocity server onto another backend server with maximum of 2 players, all 4 players can connect.

Steps/models to reproduce

Setup a network with velocity and two backend servers
set the maximum players of the other server than the lobby to e.g. 2
send more than 2 players to the backend server with the maximum players of 2

Plugin and Datapack List

image
image

[00:11:37] [Server thread/INFO]: Server Plugins (8):
[00:11:37] [Server thread/INFO]: Paper Plugins:
[00:11:37] [Server thread/INFO]: - SignedVelocity
[00:11:37] [Server thread/INFO]: Bukkit Plugins:
[00:11:37] [Server thread/INFO]: - CloudNet-Bridge, FastAsyncWorldEdit, PacketWrapper, ProtocolLib, spark, ViaBackwards, ViaVersion
[00:11:39] [Server thread/INFO]: There are 2 data pack(s) enabled: [vanilla (built-in)], [file/bukkit (world)]
[00:11:39] [Server thread/INFO]: There are no more data packs available

Paper version

[00:11:10] [Server thread/INFO]: This server is running Paper version git-Paper-496 (MC: 1.20.4) (Implementing API version 1.20.4-R0.1-SNAPSHOT) (Git: 7ac24a1 on ver/1.20.4)
You are running the latest version

Other

No response

@paaulhier paaulhier added status: needs triage type: bug Something doesn't work as it was intended to. labels May 5, 2024
@papermc-sniffer papermc-sniffer bot added the version: 1.20.4 Game version 1.20.4 label May 5, 2024
@electronicboy
Copy link
Member

Mojangs logic is generally not designed to deal with race conditions, and most solutions that I can think of that would deal with this would generally only aid in the potential to cause more harm to a server, especially with such lower player counts. I think that the general solution here would generally be to deal with that elsewhere such as on a proxy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs triage type: bug Something doesn't work as it was intended to. version: 1.20.4 Game version 1.20.4
Projects
Status: 🕑 Needs Triage
Development

No branches or pull requests

2 participants